[ https://issues.apache.org/jira/browse/HBASE-17862?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15993243#comment-15993243 ]
Chia-Ping Tsai commented on HBASE-17862: ---------------------------------------- [~lifove] Would you please close the pull request in Github? Thanks. > 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 > Assignee: JC > Priority: Trivial > Fix For: 2.0.0, 1.4.0, 1.2.6, 1.3.2, 1.1.11 > > Attachments: > 0001-HBASE-17862-Fix-a-condition-that-always-returns-true.patch > > > Hi > 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 > {code} > 209 > 210 ColumnPaginationFilter other = (ColumnPaginationFilter)o; > 211 if (this.columnOffset != null) { > 212 return this.getLimit() == this.getLimit() && > 213 Bytes.equals(this.getColumnOffset(), other.getColumnOffset()); > 214 } > {code} > It should be? > {code} > 212 return this.getLimit() == other.getLimit() && > {code} > This might be just a code smell as Bytes.equals can be enough for the return > value but wanted to report just in case. > Thanks! -- This message was sent by Atlassian JIRA (v6.3.15#6346)