[jira] [Commented] (HBASE-24588) Normalizer plan execution is not consistent between plan types
[ https://issues.apache.org/jira/browse/HBASE-24588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17147345#comment-17147345 ] Hudson commented on HBASE-24588: Results for branch master [build #1771 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/master/1771/]: (x) *{color:red}-1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/master/1771/General_20Nightly_20Build_20Report/] (x) {color:red}-1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1600//JDK8_Nightly_Build_Report_(Hadoop2)/] (x) {color:red}-1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1771/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/master/1771/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > Normalizer plan execution is not consistent between plan types > -- > > Key: HBASE-24588 > URL: https://issues.apache.org/jira/browse/HBASE-24588 > Project: HBase > Issue Type: Bug > Components: master, Normalizer >Affects Versions: 2.3.0 >Reporter: Nick Dimiduk >Assignee: Viraj Jasani >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.0 > > > I left a comment on a merged > [commit|https://github.com/apache/hbase/commit/5d0e0fc5fd09bddb2d766d1e24e28e472961f454#r39987289] > where a little discussion has blossomed. Right now the normalizer produces > two types of plans: "split" or "merge". The master receives the list of plans > and executes them in a simple loop. The way it does the actual execution is > delegated off to the plan implementation. > The bug I noticed is that the two implementations are subtly different. Both > use async APIs to submit procedures, but "split" blocks on completion while > "merge" does not. Furthermore, because "split" blocks, it's able to capture > any exception that's thrown, while "merge" cannot. > These implementations should be made consistent. My thinking at the moment is > this {{execute}} method should instead be named {{submit}}, creating and > returning the {{Future}} that represents whatever work it submitted, and the > calling context should handle the resolution of those futures in a single > place. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-24588) Normalizer plan execution is not consistent between plan types
[ https://issues.apache.org/jira/browse/HBASE-24588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17147301#comment-17147301 ] Hudson commented on HBASE-24588: Results for branch branch-2.3 [build #159 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.3/159/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.3/159/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.3/159/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.3/159/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.3/159/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > Normalizer plan execution is not consistent between plan types > -- > > Key: HBASE-24588 > URL: https://issues.apache.org/jira/browse/HBASE-24588 > Project: HBase > Issue Type: Bug > Components: master, Normalizer >Affects Versions: 2.3.0 >Reporter: Nick Dimiduk >Assignee: Viraj Jasani >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.0 > > > I left a comment on a merged > [commit|https://github.com/apache/hbase/commit/5d0e0fc5fd09bddb2d766d1e24e28e472961f454#r39987289] > where a little discussion has blossomed. Right now the normalizer produces > two types of plans: "split" or "merge". The master receives the list of plans > and executes them in a simple loop. The way it does the actual execution is > delegated off to the plan implementation. > The bug I noticed is that the two implementations are subtly different. Both > use async APIs to submit procedures, but "split" blocks on completion while > "merge" does not. Furthermore, because "split" blocks, it's able to capture > any exception that's thrown, while "merge" cannot. > These implementations should be made consistent. My thinking at the moment is > this {{execute}} method should instead be named {{submit}}, creating and > returning the {{Future}} that represents whatever work it submitted, and the > calling context should handle the resolution of those futures in a single > place. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-24588) Normalizer plan execution is not consistent between plan types
[ https://issues.apache.org/jira/browse/HBASE-24588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17147167#comment-17147167 ] Hudson commented on HBASE-24588: Results for branch branch-2 [build #2722 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2722/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2722/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2722/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2722/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2722/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > Normalizer plan execution is not consistent between plan types > -- > > Key: HBASE-24588 > URL: https://issues.apache.org/jira/browse/HBASE-24588 > Project: HBase > Issue Type: Bug > Components: master, Normalizer >Affects Versions: 2.3.0 >Reporter: Nick Dimiduk >Assignee: Viraj Jasani >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.0 > > > I left a comment on a merged > [commit|https://github.com/apache/hbase/commit/5d0e0fc5fd09bddb2d766d1e24e28e472961f454#r39987289] > where a little discussion has blossomed. Right now the normalizer produces > two types of plans: "split" or "merge". The master receives the list of plans > and executes them in a simple loop. The way it does the actual execution is > delegated off to the plan implementation. > The bug I noticed is that the two implementations are subtly different. Both > use async APIs to submit procedures, but "split" blocks on completion while > "merge" does not. Furthermore, because "split" blocks, it's able to capture > any exception that's thrown, while "merge" cannot. > These implementations should be made consistent. My thinking at the moment is > this {{execute}} method should instead be named {{submit}}, creating and > returning the {{Future}} that represents whatever work it submitted, and the > calling context should handle the resolution of those futures in a single > place. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-24588) Normalizer plan execution is not consistent between plan types
[ https://issues.apache.org/jira/browse/HBASE-24588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17146720#comment-17146720 ] Hudson commented on HBASE-24588: Results for branch master [build #1769 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/master/1769/]: (x) *{color:red}-1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/master/1769/General_20Nightly_20Build_20Report/] (x) {color:red}-1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1600//JDK8_Nightly_Build_Report_(Hadoop2)/] (x) {color:red}-1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1769/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/master/1769/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > Normalizer plan execution is not consistent between plan types > -- > > Key: HBASE-24588 > URL: https://issues.apache.org/jira/browse/HBASE-24588 > Project: HBase > Issue Type: Bug > Components: master, Normalizer >Affects Versions: 2.3.0 >Reporter: Nick Dimiduk >Assignee: Viraj Jasani >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.1, 2.4.0 > > > I left a comment on a merged > [commit|https://github.com/apache/hbase/commit/5d0e0fc5fd09bddb2d766d1e24e28e472961f454#r39987289] > where a little discussion has blossomed. Right now the normalizer produces > two types of plans: "split" or "merge". The master receives the list of plans > and executes them in a simple loop. The way it does the actual execution is > delegated off to the plan implementation. > The bug I noticed is that the two implementations are subtly different. Both > use async APIs to submit procedures, but "split" blocks on completion while > "merge" does not. Furthermore, because "split" blocks, it's able to capture > any exception that's thrown, while "merge" cannot. > These implementations should be made consistent. My thinking at the moment is > this {{execute}} method should instead be named {{submit}}, creating and > returning the {{Future}} that represents whatever work it submitted, and the > calling context should handle the resolution of those futures in a single > place. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-24588) Normalizer plan execution is not consistent between plan types
[ https://issues.apache.org/jira/browse/HBASE-24588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17146500#comment-17146500 ] Viraj Jasani commented on HBASE-24588: -- True, let me create two backport PRs for branch-2 and branch-2.3. After QA build results, we can merge them. > Normalizer plan execution is not consistent between plan types > -- > > Key: HBASE-24588 > URL: https://issues.apache.org/jira/browse/HBASE-24588 > Project: HBase > Issue Type: Bug > Components: master, Normalizer >Affects Versions: 2.3.0 >Reporter: Nick Dimiduk >Assignee: Viraj Jasani >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.1, 2.4.0 > > > I left a comment on a merged > [commit|https://github.com/apache/hbase/commit/5d0e0fc5fd09bddb2d766d1e24e28e472961f454#r39987289] > where a little discussion has blossomed. Right now the normalizer produces > two types of plans: "split" or "merge". The master receives the list of plans > and executes them in a simple loop. The way it does the actual execution is > delegated off to the plan implementation. > The bug I noticed is that the two implementations are subtly different. Both > use async APIs to submit procedures, but "split" blocks on completion while > "merge" does not. Furthermore, because "split" blocks, it's able to capture > any exception that's thrown, while "merge" cannot. > These implementations should be made consistent. My thinking at the moment is > this {{execute}} method should instead be named {{submit}}, creating and > returning the {{Future}} that represents whatever work it submitted, and the > calling context should handle the resolution of those futures in a single > place. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-24588) Normalizer plan execution is not consistent between plan types
[ https://issues.apache.org/jira/browse/HBASE-24588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17146494#comment-17146494 ] Nick Dimiduk commented on HBASE-24588: -- This is a good fix as the existing behavior is broken. +1 for branch-2.3 and 2.3.0rc1. > Normalizer plan execution is not consistent between plan types > -- > > Key: HBASE-24588 > URL: https://issues.apache.org/jira/browse/HBASE-24588 > Project: HBase > Issue Type: Bug > Components: master, Normalizer >Affects Versions: 2.3.0 >Reporter: Nick Dimiduk >Assignee: Viraj Jasani >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.1, 2.4.0 > > > I left a comment on a merged > [commit|https://github.com/apache/hbase/commit/5d0e0fc5fd09bddb2d766d1e24e28e472961f454#r39987289] > where a little discussion has blossomed. Right now the normalizer produces > two types of plans: "split" or "merge". The master receives the list of plans > and executes them in a simple loop. The way it does the actual execution is > delegated off to the plan implementation. > The bug I noticed is that the two implementations are subtly different. Both > use async APIs to submit procedures, but "split" blocks on completion while > "merge" does not. Furthermore, because "split" blocks, it's able to capture > any exception that's thrown, while "merge" cannot. > These implementations should be made consistent. My thinking at the moment is > this {{execute}} method should instead be named {{submit}}, creating and > returning the {{Future}} that represents whatever work it submitted, and the > calling context should handle the resolution of those futures in a single > place. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-24588) Normalizer plan execution is not consistent between plan types
[ https://issues.apache.org/jira/browse/HBASE-24588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17146475#comment-17146475 ] Viraj Jasani commented on HBASE-24588: -- Thank you [~ndimiduk] for the review. I believe we can have this in 2.3, but I am fine with 2.3.1 also, you have already taken so many such last moment Jiras and I don't want to give more headache :) So let me keep this till branch-2 for now? > Normalizer plan execution is not consistent between plan types > -- > > Key: HBASE-24588 > URL: https://issues.apache.org/jira/browse/HBASE-24588 > Project: HBase > Issue Type: Bug > Components: master, Normalizer >Affects Versions: 2.3.0 >Reporter: Nick Dimiduk >Assignee: Viraj Jasani >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.1, 2.4.0 > > > I left a comment on a merged > [commit|https://github.com/apache/hbase/commit/5d0e0fc5fd09bddb2d766d1e24e28e472961f454#r39987289] > where a little discussion has blossomed. Right now the normalizer produces > two types of plans: "split" or "merge". The master receives the list of plans > and executes them in a simple loop. The way it does the actual execution is > delegated off to the plan implementation. > The bug I noticed is that the two implementations are subtly different. Both > use async APIs to submit procedures, but "split" blocks on completion while > "merge" does not. Furthermore, because "split" blocks, it's able to capture > any exception that's thrown, while "merge" cannot. > These implementations should be made consistent. My thinking at the moment is > this {{execute}} method should instead be named {{submit}}, creating and > returning the {{Future}} that represents whatever work it submitted, and the > calling context should handle the resolution of those futures in a single > place. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-24588) Normalizer plan execution is not consistent between plan types
[ https://issues.apache.org/jira/browse/HBASE-24588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17140324#comment-17140324 ] Viraj Jasani commented on HBASE-24588: -- [~ndimiduk] If you already had some patch or started working on this, please let me know, I will unassign myself. If not, let me try to work on this and see if I can make a patch ready for review. > Normalizer plan execution is not consistent between plan types > -- > > Key: HBASE-24588 > URL: https://issues.apache.org/jira/browse/HBASE-24588 > Project: HBase > Issue Type: Bug > Components: master, Normalizer >Affects Versions: 2.3.0 >Reporter: Nick Dimiduk >Assignee: Viraj Jasani >Priority: Major > > I left a comment on a merged > [commit|https://github.com/apache/hbase/commit/5d0e0fc5fd09bddb2d766d1e24e28e472961f454#r39987289] > where a little discussion has blossomed. Right now the normalizer produces > two types of plans: "split" or "merge". The master receives the list of plans > and executes them in a simple loop. The way it does the actual execution is > delegated off to the plan implementation. > The bug I noticed is that the two implementations are subtly different. Both > use async APIs to submit procedures, but "split" blocks on completion while > "merge" does not. Furthermore, because "split" blocks, it's able to capture > any exception that's thrown, while "merge" cannot. > These implementations should be made consistent. My thinking at the moment is > this {{execute}} method should instead be named {{submit}}, creating and > returning the {{Future}} that represents whatever work it submitted, and the > calling context should handle the resolution of those futures in a single > place. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-24588) Normalizer plan execution is not consistent between plan types
[ https://issues.apache.org/jira/browse/HBASE-24588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17140320#comment-17140320 ] Viraj Jasani commented on HBASE-24588: -- {quote}the calling context should handle the resolution of those futures in a single place. {quote} HMaster should call future.get() for all tables after plans are submitted for all tables? That might be better right? And if polling throws any Exception, we log it only at one place: HMaster and we maintain a count say splitPlanFailedCount, mergePlanFailedCount and if it is helpful, maybe expose them as metrics? Thought? > Normalizer plan execution is not consistent between plan types > -- > > Key: HBASE-24588 > URL: https://issues.apache.org/jira/browse/HBASE-24588 > Project: HBase > Issue Type: Bug > Components: master, Normalizer >Affects Versions: 2.3.0 >Reporter: Nick Dimiduk >Priority: Major > > I left a comment on a merged > [commit|https://github.com/apache/hbase/commit/5d0e0fc5fd09bddb2d766d1e24e28e472961f454#r39987289] > where a little discussion has blossomed. Right now the normalizer produces > two types of plans: "split" or "merge". The master receives the list of plans > and executes them in a simple loop. The way it does the actual execution is > delegated off to the plan implementation. > The bug I noticed is that the two implementations are subtly different. Both > use async APIs to submit procedures, but "split" blocks on completion while > "merge" does not. Furthermore, because "split" blocks, it's able to capture > any exception that's thrown, while "merge" cannot. > These implementations should be made consistent. My thinking at the moment is > this {{execute}} method should instead be named {{submit}}, creating and > returning the {{Future}} that represents whatever work it submitted, and the > calling context should handle the resolution of those futures in a single > place. -- This message was sent by Atlassian Jira (v8.3.4#803005)