[ 
https://issues.apache.org/jira/browse/ACCUMULO-4376?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15443113#comment-15443113
 ] 

Mario Pastorelli commented on ACCUMULO-4376:
--------------------------------------------

Well [~ctubbsii], that's disappointing. You do have implemented this in 
ACCUMULO-2589 even if there is nothing pointing to this JIRA or anything 
related in ACCUMULO-2589. [~elserj] I guess we can close this as duplicate to 
avoid wasting other developers time.

[~ctubbsii] [~dlmarion] I don't wish to get started for the simple reason that 
if I use a huge branch like ACCUMULO-2589 as starting point, I would have to 
lookup every single related change over 342 files changed from 1.8 (4.665 lines 
added, 10.137 lines deleted where most of the commits messages are "Merge 
branch '1.8'", github gives up in showing the changes), understand if they are 
related to this JIRA, understand why and what has been changed from the current 
design, and then maybe implement this again. In the meanwhile everything has 
been already implemented and I don't like pointless exercises.
Besides, I miss where all those important design decisions have been taken and 
I don't understand why there are JIRAs desynchronized with the work in that 
branch.

[~elserj] let me know if I can help with something else.

> Introduce a Builders for "data" classes
> ---------------------------------------
>
>                 Key: ACCUMULO-4376
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4376
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: client
>            Reporter: Josh Elser
>             Fix For: 2.0.0
>
>
> In looking at ACCUMULO-4375, I was a little frustrated at how we have 3x 
> constructors than Key really provides just to support {{byte[]}}, {{Text}}, 
> and {{CharSequence}} arguments. Additionally, the {{copy}} argument forces 
> the user to use the most specific (most arguments) constructor if they want 
> to avoid the copy. This makes constructing a Key from just a row while 
> avoiding a copy very pedantic.
> I think a KeyBuilder (or KeyFactory) class would be a big usability benefit 
> and reduce the amount of code that clients have to write to most efficiently 
> construct Keys.



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

Reply via email to