You don't need Iterator<IndexCount> iterator() as we have forEachCount( BitCountConsumer )
I guess we need something like add( Iterator<IndexCount>) or add( Collection<IndexCount> ) or add( Stream<IndexCount> ) It would be nice if we could have a BitCountProducer class that we could just pass to an add() method. On Wed, Mar 18, 2020 at 11:50 AM Alex Herbert <alex.d.herb...@gmail.com> wrote: > > > > On 18 Mar 2020, at 11:14, Claude Warren <cla...@xenei.com> wrote: > > > > On a slightly different note. CountingBloomFilters have no way to > perform > > a reload. All other bloom filters you can dump the bits and reload > > (trivial) but if you preserve the counts from a bloom filter and want to > > reload them you can't. We need a constructor that takes the index,count > > pairs somehow. > > Iterator<int[]> ? > > Or foolproof: > > class IndexCount { > final int index; > final int count; > // ... > } > > Iterator<IndexCount> > > > The CountingBloomFilter already has a method forEachCount(…). > > I was reluctant to add some sort of iterator: > > Iterator<?> iterator() > > But we could put in: > > Iterator<IndexCount> iterator() > > It would be inefficient but at least it is fool-proof. The operation is > unlikely to be used very often. > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > > -- I like: Like Like - The likeliest place on the web <http://like-like.xenei.com> LinkedIn: http://www.linkedin.com/in/claudewarren