[jira] [Comment Edited] (ACCUMULO-1919) Size of visibility cache in AccumuloFileOutputFormat should be configurable

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

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

Josh Elser edited comment on ACCUMULO-1919 at 6/6/14 3:28 PM:
--------------------------------------------------------------

Great! This LGTM. I'll run some tests with the patch applied, and then apply it if later today if no one else has any opinions/requests.

Actually, it doesn't compile. It looks like you're missing an import:

{noformat}
[ERROR] /.../accumulo-1.6.git/core/src/test/java/org/apache/accumulo/core/client/mapred/AccumuloFileOutputFormatTest.java:[149,7] cannot find symbol
  symbol:   variable ConfiguratorBase
  location: class org.apache.accumulo.core.client.mapred.AccumuloFileOutputFormatTest.MRTester
{noformat}


was (Author: elserj):
Great! This LGTM. I'll run some tests with the patch applied, and then apply it if later today if no one else has any opinions/requests.

> Size of visibility cache in AccumuloFileOutputFormat should be configurable
> ---------------------------------------------------------------------------
>
>                 Key: ACCUMULO-1919
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-1919
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: client
>    Affects Versions: 1.5.0, 1.6.0
>            Reporter: Mike Drob
>            Priority: Minor
>              Labels: newbie
>         Attachments: ACCUMULO-1919.patch, ACCUMULO-1919.patch
>
>
> AccumuloFileOutputFormat uses an LRUMap hard-coded to max size of 1000. It is possible that the client writing the data knows more about the visibility distribution and can specify a better value if this is made configurable.



--
This message was sent by Atlassian JIRA
(v6.2#6252)