accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christopher Tubbs (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-1488) support BigDecimal encoding for basic built-in combiners
Date Sat, 15 Jun 2013 16:19:20 GMT


Christopher Tubbs commented on ACCUMULO-1488:

[~afuchs], it seems to me, if I'm understanding your points correctly, then it seems your
justification can be summarized as "The number of users who will find these useful to have
in the 1.5 line is non-zero, the costs to support these particular users is less if we include
it in the 1.5.1 release, and the risks are minimal, so why not?"

If we ignore, for the moment, the issue of this practice setting a precedent (or solidifying
one that perhaps began with the proxy and MiniAccumuloCluster, whose support costs raised
the question of whether we should avoid this practice in the first place), I have no significant
objection to this reasoning on this particular feature.

However, I do have an objection to this kind of justification as a general rule to the practice
of back-porting, so perhaps we can continue the discussion of establishing a policy to address
this practice more efficiently in the future, outside the narrow scope of this particular
ticket. If we come to a consensus in that arena, we can revisit here.

(Also, I'm not sure the point of mentioning that the patch existed before committing has any
bearing on this exchange, since the issue we're specifically addressing is which branches
the feature was applied to, not the feature itself. I mention this now, in case it does apply
in some way which I did not understand, then you have an opportunity to clarify its applicability.)

In any case, I thank you for your patience and continued responsiveness to my objections.
I know these conversations can seem to drag on longer than we'd like, but I think it's a necessary
part of this type of collaborative environment.
> support BigDecimal encoding for basic built-in combiners
> --------------------------------------------------------
>                 Key: ACCUMULO-1488
>                 URL:
>             Project: Accumulo
>          Issue Type: New Feature
>            Reporter: Adam Fuchs
>             Fix For: 1.5.1, 1.6.0
>         Attachments: ACCUMULO-1488.patch
> Support sum, min, and max operations for decimal numbers using the java.math.BigDecimal
String encoding and decoding functions (BigDecimal.toString() and BigDecimal(String)).

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

View raw message