[ACCUMULO-4377] Document and add test for Key constructors copy behavior

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Duplicate
• Assignee: Unassigned
• Reporter: Keith Turner
• Created: 18/Jul/16 12:37
• Updated: 03/Feb/20 20:42
• Resolved: 03/Feb/20 20:42

Description

While looking at Github PR #125 I thought it was nice that the new constructors documented the copy behavior. It would be nice to update the javadoc for the constructors that existed before the PR to mention the copy behavior.

Also all of the constructors copy behavior should be tested in KeyTest.java. Should do things like pass in byte array to constructor, modify bye array, verify Key does not reflect changes.