It looks like this is part of ACCUMULO-517? 23 does seem pretty arbitrary. I'm guessing Keith chose that number after some experimentation to ensure it was low enough to keep the counters up-to-date, but high enough to avoid a lot of synchronization overhead.
-- Christopher L Tubbs II http://gravatar.com/ctubbsii On Tue, Sep 3, 2013 at 9:29 PM, David Medinets <[email protected]> wrote: > At the risk of having overlooked something obvious, I was curious about the > occurrence of 23 in the StatsIterator code: > > public void next() throws IOException { > super.next(); > numRead++; > > if (numRead % 23 == 0) { > readCounter.addAndGet(numRead); > numRead = 0; > } > } > > I'm also wondering about the StatsIterator in general. The member variables > are all private. They are incremented but never used.
