Description
In my company we use Key built directly from byte[] instead of Text. Currently Key has many constructors working with Text and only few working with byte[]. You can still use the Text-based constructor to create a Key from a byte[] by wrapping it into a Text, but this requires to box a byte[] into Text without any good reason.
I propose to add the missing byte[]-based Key constructors, which are:
Key(byte[] row) Key(byte[] row, long ts) Key(byte[] row, byte[] cf) Key(byte[] row, byte[] cf, byte[] cq) Key(byte[] row, byte[] cf, byte[] cq, byte[] cv) Key(byte[] row, byte[] cf, byte[] cq, long ts) Key(byte[] row, byte[] cf, byte[] cq, ColumnVisibility cv, long ts)
The new constructor should behave like the Text-based counterpart, for instance:
byte[] row = new byte[] {0}; assertEquals(new Key(row), new Key(new Text(row)));
Attachments
Issue Links
- is related to
-
ACCUMULO-4376 Introduce a Builders for "data" classes
-
- Resolved
-
- links to
1.
|
New Key Constructors need @since tags |
|
Resolved | Mario Pastorelli |
|
