[jira] [Resolved] (ACCUMULO-3364) Inconsistency between java and native map update within single mutation

["Josh Elser (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/ACCUMULO-3364?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Josh Elser resolved ACCUMULO-3364.
----------------------------------
       Resolution: Duplicate
    Fix Version/s:     (was: 1.6.6)
                       (was: 1.7.2)
                       (was: 1.8.0)

Closing as duplicate of ACCUMULO-4148

> Inconsistency between java and native map update within single mutation
> -----------------------------------------------------------------------
>
>                 Key: ACCUMULO-3364
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3364
>             Project: Accumulo
>          Issue Type: Bug
>          Components: tserver
>    Affects Versions: 1.6.1
>            Reporter: Josh Elser
>            Priority: Critical
>
> Noticed the following when I was poking around with {{ExamplesIT#testStatsCombiner}}:
> {code}
>     String table = getUniqueNames(1)[0];
>     c.tableOperations().create(table);
>     is = new IteratorSetting(10, StatsCombiner.class);
>     StatsCombiner.setCombineAllColumns(is, true);
>     c.tableOperations().attachIterator(table, is);
>     bw = c.createBatchWriter(table, bwc);
>     Mutation m = new Mutation("foo");
>     m.put("a", "b", "1");
>     m.put("a", "b", "3");
>     bw.addMutation(m);
>     bw.flush();
>     Iterator<Entry<Key,Value>> iter = c.createScanner(table, Authorizations.EMPTY).iterator();
>     assertTrue("Iterator had no results", iter.hasNext());
>     Entry<Key,Value> e = iter.next();
>     assertEquals("Results ", "1,3,4,2", e.getValue().toString());
> {code}
> When run against the Java maps, the above succeeds as the test expects it to. However, this fails when the instance is using native maps.
> Talked to [~kturner] about this, and he noted that the Java maps [use a one-up counter when performing updates|https://github.com/apache/accumulo/blob/1.6.1/server/tserver/src/main/java/org/apache/accumulo/tserver/InMemoryMap.java#L404] whereas it does not appear that the NativeMap does this.
> The reason I got stuck on it is because I expected to see both updates because the StatsCombiner was running below the VersioningIterator, but I would only see a single Key-Value out of the scanner (a value of "3,3,3,1").
> My gut reaction is that we should have the same semantics across both Java and Native maps, although I'm not sure which one is correct/expected yet.



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