hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "JC (JIRA)" <j...@apache.org>
Subject [jira] [Created] (HBASE-17862) Condition that always returns true
Date Fri, 31 Mar 2017 20:28:42 GMT
JC created HBASE-17862:

             Summary: Condition that always returns true
                 Key: HBASE-17862
                 URL: https://issues.apache.org/jira/browse/HBASE-17862
             Project: HBase
          Issue Type: Bug
          Components: Client
            Reporter: JC
            Priority: Trivial


In recent github mirror of hbase, I've found the following code smell.

Path: hbase-client/src/main/java/org/apache/hadoop/hbase/filter/ColumnPaginationFilter.java

210     ColumnPaginationFilter other = (ColumnPaginationFilter)o;
211     if (this.columnOffset != null) {
212       return this.getLimit() == this.getLimit() &&
213           Bytes.equals(this.getColumnOffset(), other.getColumnOffset());
214     }

It should be?
212       return this.getLimit() == other.getLimit() &&

This might be just a code smell as Bytes.equals can be enough for the return value but wanted
to report just in case.


This message was sent by Atlassian JIRA

View raw message