Mailing-List: contact notifications-help@accumulo.apache.org; run by ezmlm
Precedence: bulk
Reply-To: jira@apache.org
Date: Sat, 16 Jul 2016 22:12:20 +0000 (UTC)
From: "Mario Pastorelli (JIRA)" <jira@apache.org>
To: notifications@accumulo.apache.org
Message-ID: <JIRA.12990134.1468707105000.51391.1468707140420@Atlassian.JIRA>
In-Reply-To: <JIRA.12990134.1468707105000@Atlassian.JIRA>
References: <JIRA.12990134.1468707105000@Atlassian.JIRA> <JIRA.12990134.1468707105593@arcas>
Subject: [jira] [Created] (ACCUMULO-4375) Add missing Key constructors
 taking array of bytes as argument
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Sat, 16 Jul 2016 22:12:22 -0000

Mario Pastorelli created ACCUMULO-4375:
------------------------------------------

             Summary: Add missing Key constructors taking array of bytes as argument
                 Key: ACCUMULO-4375
                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4375
             Project: Accumulo
          Issue Type: Improvement
          Components: core
            Reporter: Mario Pastorelli


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:

{code:java}
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)
{code}

The new constructor should behave like the {{Text}}-based counterpart, for instance:

{code:java}
byte[] row = new byte[] {0};
assertEquals(new Key(row), new Key(new Text(row)));
{code}


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)