hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andrew Purtell (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (HBASE-19194) TestRSGroupsBase has some always false checks
Date Wed, 08 Nov 2017 01:03:00 GMT

    [ https://issues.apache.org/jira/browse/HBASE-19194?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16243202#comment-16243202
] 

Andrew Purtell edited comment on HBASE-19194 at 11/8/17 1:02 AM:
-----------------------------------------------------------------

RSGroups tests with changes here pass on master / branch-2:

{noformat}
[INFO] Running org.apache.hadoop.hbase.master.balancer.TestRSGroupBasedLoadBalancer
[INFO] Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.694 s - in org.apache.hadoop.hbase.master.balancer.TestRSGroupBasedLoadBalancer
[INFO] Running org.apache.hadoop.hbase.rsgroup.TestRSGroups
[INFO] Tests run: 20, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 120.148 s - in org.apache.hadoop.hbase.rsgroup.TestRSGroups
[INFO] Running org.apache.hadoop.hbase.rsgroup.TestRSGroupsOfflineMode
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 25.473 s - in org.apache.hadoop.hbase.rsgroup.TestRSGroupsOfflineMode
{noformat}




was (Author: apurtell):
RSGroups with updated tests pass on master / branch-2:

{noformat}
[INFO] Running org.apache.hadoop.hbase.master.balancer.TestRSGroupBasedLoadBalancer
[INFO] Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.694 s - in org.apache.hadoop.hbase.master.balancer.TestRSGroupBasedLoadBalancer
[INFO] Running org.apache.hadoop.hbase.rsgroup.TestRSGroups
[INFO] Tests run: 20, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 120.148 s - in org.apache.hadoop.hbase.rsgroup.TestRSGroups
[INFO] Running org.apache.hadoop.hbase.rsgroup.TestRSGroupsOfflineMode
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 25.473 s - in org.apache.hadoop.hbase.rsgroup.TestRSGroupsOfflineMode
{noformat}



> TestRSGroupsBase has some always false checks
> ---------------------------------------------
>
>                 Key: HBASE-19194
>                 URL: https://issues.apache.org/jira/browse/HBASE-19194
>             Project: HBase
>          Issue Type: Bug
>          Components: rsgroup, test
>            Reporter: Mike Drob
>            Assignee: Andrew Purtell
>            Priority: Blocker
>             Fix For: 3.0.0, 1.4.0, 1.5.0, 2.0.0-beta-1
>
>         Attachments: HBASE-19194-branch-1.patch, HBASE-19194.patch
>
>
> Discovered via error-prone analysis:
> {noformat}
> [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.6.1:testCompile
(default-testCompile) on project hbase-rsgroup: Compilation failure: Compilation failure:
> [ERROR] /Users/mdrob/IdeaProjects/hbase/hbase-rsgroup/src/test/java/org/apache/hadoop/hbase/rsgroup/TestRSGroupsBase.java:[421,40]
[CollectionIncompatibleType] Argument 'region' should not be passed to this method; its type
RegionInfo is not compatible with its collection's type argument String
> [ERROR]     (see http://errorprone.info/bugpattern/CollectionIncompatibleType)
> [ERROR] /Users/mdrob/IdeaProjects/hbase/hbase-rsgroup/src/test/java/org/apache/hadoop/hbase/rsgroup/TestRSGroupsBase.java:[531,65]
[CollectionIncompatibleType] Argument 'targetRegion' should not be passed to this method;
its type String is not compatible with its collection's type argument RegionInfo
> [ERROR]     (see http://errorprone.info/bugpattern/CollectionIncompatibleType)
> {noformat}
> Both of these are checks to {{List<T>.contains}} with an element of the wrong type
so they will always return false. The first is an optimization that we can probably live without
(or short circuit on the Master, at least) the second is an assertion that will never fail,
rendering the test ineffective.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message