[HBASE-6458] new comparator twice in checkAndPut, just reuse the first one

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.94.1
• Fix Version/s: 0.94.2
• Assignee: Unassigned
• Reporter: Lijin Bin
• Created: 26/Jul/12 08:51
• Updated: 24/Oct/12 05:40
• Resolved: 25/Aug/12 00:09

Description

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.