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

Hudson commented on HBASE-6458:
-------------------------------

Integrated in HBase-0.94-security-on-Hadoop-23 #7 (See 
[https://builds.apache.org/job/HBase-0.94-security-on-Hadoop-23/7/])
    HBASE-6458 new comparator twice in checkAndPut, just reuse the first one 
(binlijin) (Revision 1377158)

     Result = FAILURE
larsh : 
Files : 
* 
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java

                
> new comparator twice in checkAndPut, just reuse the first one
> -------------------------------------------------------------
>
>                 Key: HBASE-6458
>                 URL: https://issues.apache.org/jira/browse/HBASE-6458
>             Project: HBase
>          Issue Type: Improvement
>    Affects Versions: 0.94.1
>            Reporter: binlijin
>             Fix For: 0.94.2
>
>         Attachments: HBASE-6458-0.94-2.patch, HBASE-6458-0.94.patch
>
>
>   public boolean checkAndPut(final byte[] regionName, final byte[] row,
>       final byte[] family, final byte[] qualifier, final byte[] value,
>       final Put put) throws IOException {
>              WritableByteArrayComparable comparator = new 
> BinaryComparator(value);
>                  boolean result = checkAndMutate(regionName, row, family, 
> qualifier,
>                               CompareOp.EQUAL, new BinaryComparator(value), 
> put,lock);
>   }
>   The comparator creates twice, because the comparator is not changed, so we 
> can reuse the comparator, the second is useless. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to