Sanjeet Malhotra created HBASE-29278:
----------------------------------------

             Summary: Replace isEmptyColumn in CompactionScanner with 
equivalent method from HBase CellUtil
                 Key: HBASE-29278
                 URL: https://issues.apache.org/jira/browse/HBASE-29278
             Project: HBase
          Issue Type: Improvement
            Reporter: Sanjeet Malhotra


Currently we have following utility method defined in Phoenix:
{code:java}
public static boolean isEmptyColumn(Cell cell, byte[] emptyCF, byte[] emptyCQ) {
        return CellUtil.matchingFamily(cell, emptyCF, 0, emptyCF.length) &&
               CellUtil.matchingQualifier(cell, emptyCQ, 0, emptyCQ.length);
} {code}
Firstly this method gives an impression based on its name that it knows about 
emptyCF and emptyCQ while it is only comparing column family and column 
qualifier against provided `emptyCF` and `emptyCQ`.

Also, there is an equivalent method in HBase CellUtil:
{code:java}
public static boolean matchingColumn(final Cell left, final byte[] fam, final 
byte[] qual) {
  return matchingFamily(left, fam) && matchingQualifier(left, qual);
} {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to