[jira] [Updated] (HBASE-18981) Address issues found by error-prone in hbase-client
[ https://issues.apache.org/jira/browse/HBASE-18981?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Biju Nair updated HBASE-18981: -- Labels: quota (was: ) > Address issues found by error-prone in hbase-client > --- > > Key: HBASE-18981 > URL: https://issues.apache.org/jira/browse/HBASE-18981 > Project: HBase > Issue Type: Sub-task >Reporter: Mike Drob >Assignee: Mike Drob >Priority: Trivial > Labels: quota > Fix For: 2.0.0-alpha-4, 2.0.0 > > Attachments: HBASE-18981.patch > > > I thought this was something that we would already catch in findbugs, but > maybe not? > {noformat} > [ERROR] Failed to execute goal > org.apache.maven.plugins:maven-compiler-plugin:3.6.1:compile > (default-compile) on project hbase-client: Compilation failure > [ERROR] > /Users/mdrob/IdeaProjects/hbase/hbase-client/src/main/java/org/apache/hadoop/hbase/shaded/protobuf/ProtobufUtil.java:[2476,5] > [MissingCasesInEnumSwitch] Non-exhaustive switch; either add a default or > handle the remaining cases: GLOBAL_BYPASS > [ERROR] (see http://errorprone.info/bugpattern/MissingCasesInEnumSwitch) > {noformat} > cc: [~elserj], this looks like it is part of quota code. I'm happy to add a > trivial patch to error on the missing case, but want somebody with more > contextual knowledge to look at it. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-18981) Address issues found by error-prone in hbase-client
[ https://issues.apache.org/jira/browse/HBASE-18981?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mike Drob updated HBASE-18981: -- Resolution: Fixed Fix Version/s: (was: 2.0.0-beta-1) 2.0.0-alpha-4 Status: Resolved (was: Patch Available) > Address issues found by error-prone in hbase-client > --- > > Key: HBASE-18981 > URL: https://issues.apache.org/jira/browse/HBASE-18981 > Project: HBase > Issue Type: Sub-task >Reporter: Mike Drob >Assignee: Mike Drob > Fix For: 3.0.0, 2.0.0-alpha-4 > > Attachments: HBASE-18981.patch > > > I thought this was something that we would already catch in findbugs, but > maybe not? > {noformat} > [ERROR] Failed to execute goal > org.apache.maven.plugins:maven-compiler-plugin:3.6.1:compile > (default-compile) on project hbase-client: Compilation failure > [ERROR] > /Users/mdrob/IdeaProjects/hbase/hbase-client/src/main/java/org/apache/hadoop/hbase/shaded/protobuf/ProtobufUtil.java:[2476,5] > [MissingCasesInEnumSwitch] Non-exhaustive switch; either add a default or > handle the remaining cases: GLOBAL_BYPASS > [ERROR] (see http://errorprone.info/bugpattern/MissingCasesInEnumSwitch) > {noformat} > cc: [~elserj], this looks like it is part of quota code. I'm happy to add a > trivial patch to error on the missing case, but want somebody with more > contextual knowledge to look at it. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Updated] (HBASE-18981) Address issues found by error-prone in hbase-client
[ https://issues.apache.org/jira/browse/HBASE-18981?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mike Drob updated HBASE-18981: -- Priority: Trivial (was: Major) > Address issues found by error-prone in hbase-client > --- > > Key: HBASE-18981 > URL: https://issues.apache.org/jira/browse/HBASE-18981 > Project: HBase > Issue Type: Sub-task >Reporter: Mike Drob >Assignee: Mike Drob >Priority: Trivial > Fix For: 3.0.0, 2.0.0-alpha-4 > > Attachments: HBASE-18981.patch > > > I thought this was something that we would already catch in findbugs, but > maybe not? > {noformat} > [ERROR] Failed to execute goal > org.apache.maven.plugins:maven-compiler-plugin:3.6.1:compile > (default-compile) on project hbase-client: Compilation failure > [ERROR] > /Users/mdrob/IdeaProjects/hbase/hbase-client/src/main/java/org/apache/hadoop/hbase/shaded/protobuf/ProtobufUtil.java:[2476,5] > [MissingCasesInEnumSwitch] Non-exhaustive switch; either add a default or > handle the remaining cases: GLOBAL_BYPASS > [ERROR] (see http://errorprone.info/bugpattern/MissingCasesInEnumSwitch) > {noformat} > cc: [~elserj], this looks like it is part of quota code. I'm happy to add a > trivial patch to error on the missing case, but want somebody with more > contextual knowledge to look at it. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Updated] (HBASE-18981) Address issues found by error-prone in hbase-client
[ https://issues.apache.org/jira/browse/HBASE-18981?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mike Drob updated HBASE-18981: -- Fix Version/s: 3.0.0 > Address issues found by error-prone in hbase-client > --- > > Key: HBASE-18981 > URL: https://issues.apache.org/jira/browse/HBASE-18981 > Project: HBase > Issue Type: Sub-task >Reporter: Mike Drob >Assignee: Mike Drob > Fix For: 3.0.0, 2.0.0-beta-1 > > Attachments: HBASE-18981.patch > > > I thought this was something that we would already catch in findbugs, but > maybe not? > {noformat} > [ERROR] Failed to execute goal > org.apache.maven.plugins:maven-compiler-plugin:3.6.1:compile > (default-compile) on project hbase-client: Compilation failure > [ERROR] > /Users/mdrob/IdeaProjects/hbase/hbase-client/src/main/java/org/apache/hadoop/hbase/shaded/protobuf/ProtobufUtil.java:[2476,5] > [MissingCasesInEnumSwitch] Non-exhaustive switch; either add a default or > handle the remaining cases: GLOBAL_BYPASS > [ERROR] (see http://errorprone.info/bugpattern/MissingCasesInEnumSwitch) > {noformat} > cc: [~elserj], this looks like it is part of quota code. I'm happy to add a > trivial patch to error on the missing case, but want somebody with more > contextual knowledge to look at it. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Updated] (HBASE-18981) Address issues found by error-prone in hbase-client
[ https://issues.apache.org/jira/browse/HBASE-18981?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mike Drob updated HBASE-18981: -- Attachment: HBASE-18981.patch Thanks, Josh! Attaching the trivial fix. > Address issues found by error-prone in hbase-client > --- > > Key: HBASE-18981 > URL: https://issues.apache.org/jira/browse/HBASE-18981 > Project: HBase > Issue Type: Sub-task >Reporter: Mike Drob > Fix For: 2.0.0-beta-1 > > Attachments: HBASE-18981.patch > > > I thought this was something that we would already catch in findbugs, but > maybe not? > {noformat} > [ERROR] Failed to execute goal > org.apache.maven.plugins:maven-compiler-plugin:3.6.1:compile > (default-compile) on project hbase-client: Compilation failure > [ERROR] > /Users/mdrob/IdeaProjects/hbase/hbase-client/src/main/java/org/apache/hadoop/hbase/shaded/protobuf/ProtobufUtil.java:[2476,5] > [MissingCasesInEnumSwitch] Non-exhaustive switch; either add a default or > handle the remaining cases: GLOBAL_BYPASS > [ERROR] (see http://errorprone.info/bugpattern/MissingCasesInEnumSwitch) > {noformat} > cc: [~elserj], this looks like it is part of quota code. I'm happy to add a > trivial patch to error on the missing case, but want somebody with more > contextual knowledge to look at it. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Updated] (HBASE-18981) Address issues found by error-prone in hbase-client
[ https://issues.apache.org/jira/browse/HBASE-18981?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mike Drob updated HBASE-18981: -- Assignee: Mike Drob Status: Patch Available (was: Open) > Address issues found by error-prone in hbase-client > --- > > Key: HBASE-18981 > URL: https://issues.apache.org/jira/browse/HBASE-18981 > Project: HBase > Issue Type: Sub-task >Reporter: Mike Drob >Assignee: Mike Drob > Fix For: 2.0.0-beta-1 > > Attachments: HBASE-18981.patch > > > I thought this was something that we would already catch in findbugs, but > maybe not? > {noformat} > [ERROR] Failed to execute goal > org.apache.maven.plugins:maven-compiler-plugin:3.6.1:compile > (default-compile) on project hbase-client: Compilation failure > [ERROR] > /Users/mdrob/IdeaProjects/hbase/hbase-client/src/main/java/org/apache/hadoop/hbase/shaded/protobuf/ProtobufUtil.java:[2476,5] > [MissingCasesInEnumSwitch] Non-exhaustive switch; either add a default or > handle the remaining cases: GLOBAL_BYPASS > [ERROR] (see http://errorprone.info/bugpattern/MissingCasesInEnumSwitch) > {noformat} > cc: [~elserj], this looks like it is part of quota code. I'm happy to add a > trivial patch to error on the missing case, but want somebody with more > contextual knowledge to look at it. -- This message was sent by Atlassian JIRA (v6.4.14#64029)