accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Keith Turner (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-1909) Adding list of conditions to conditional mutation is cumbersome.
Date Wed, 20 Nov 2013 16:29:35 GMT

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

Keith Turner commented on ACCUMULO-1909:
----------------------------------------

bq.  Why not just issue a WARN and act as a pass-through when there are no conditions present?

Two reasons : 

# The users intent was probably to have a condition, not having one indicates an error in
their code.  If warning is logged and no one see its then this bug goes unnoticed.   
# The conditional writer is an inefficient way to write mutations relative to the batch writer.

> Adding list of conditions to conditional mutation is cumbersome.
> ----------------------------------------------------------------
>
>                 Key: ACCUMULO-1909
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-1909
>             Project: Accumulo
>          Issue Type: Sub-task
>          Components: client, tserver
>            Reporter: Keith Turner
>             Fix For: 1.6.0
>
>
> The conditional mutation constructor requires that a user pass in at least one condition.
 For cases where I have a list of data that I want to create conditions from I have found
this to be very cumbersome writing code like the following.
> {code:java}
>   List<String> cdata = ...;
>   ConditionalMutation cm = null;
>   for(String s : cdata){
>     Condition condition = createCondition(s);
>      if(cm == null)
>         cm = new ConditionalMutation(row, condition)
>     else
>         cm.addCondition(condition);
>    }
> {code}



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Mime
View raw message