[jira] [Updated] (HBASE-18981) Address issues found by error-prone in hbase-client

2019-04-22 Thread Biju Nair (JIRA)


 [ 
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

2017-10-10 Thread Mike Drob (JIRA)

 [ 
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

2017-10-10 Thread Mike Drob (JIRA)

 [ 
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

2017-10-10 Thread Mike Drob (JIRA)

 [ 
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

2017-10-10 Thread Mike Drob (JIRA)

 [ 
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

2017-10-10 Thread Mike Drob (JIRA)

 [ 
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)