Sean Busbey created ACCUMULO-2511:
-------------------------------------

             Summary: Value allows equals(byte[])
                 Key: ACCUMULO-2511
                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2511
             Project: Accumulo
          Issue Type: Improvement
            Reporter: Sean Busbey


Right now the equals(Object) method for Value is

{noformat}
 @Override
  public boolean equals(Object right_obj) {
    if (right_obj instanceof byte[]) {
      return compareTo((byte[]) right_obj) == 0;
    }
    if (right_obj instanceof Value) {
      return compareTo(right_obj) == 0;
    }
    return false;
  }

{noformat}

It's in a section that says it was copied from BytesWritable.

If we're not using this optimization anywhere, I'd rather remove it since it is 
non-intuitive.

If we are using it, I'd prefer we move it into something other than the general 
equals



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to