Closes #210. Making sure maxLatency is not set to -1 by default. This ti...

...cket will be followed on by an API change to get rid of StoreConfig and use BatchWriterConfig and BatchScanConfig
I'm not sure I agree holding in data in memory for 2min before sync'ing to
the server is a reasonable default option for this. It looks like
Christopher Tubbs set this default originally. I'll reach out to him and
try to get clarity on the reasoning behind this choice.


On Mon, Sep 1, 2014 at 1:22 PM, eawagner <notifications@github.com> wrote:

> In
> commons/src/main/java/org/calrissian/accumulorecipes/commons/domain/StoreConfig.java:
>
> > @@ -36,7 +36,7 @@ public StoreConfig() {
> >       * @param maxQueryThreads the number of concurrent threads to spawn for querying
> >       */
> >      public StoreConfig(int maxQueryThreads) {
> > -        this(maxQueryThreads, 10000, -1, 1);
> > +        this(maxQueryThreads, 10000, 10000, 1);
>
> I know these defaults where arbitrary, but why not use the defaults that
> BatchWriterConfig uses?
>
> I think for maxLatency, it uses 2 mins (120000 ms).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/calrissian/accumulo-recipes/pull/215/files#r16961621>.
>