commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Claude Warren <cla...@xenei.com>
Subject [collections] ProtoBloomFilter.Builder -- opinions sought
Date Mon, 07 Oct 2019 17:01:30 GMT
Greetings,

 I am preparing a pull request to bring Bloom Filters in the the
collections package.  The pull requests is at
https://github.com/apache/commons-collections/pull/83 and the Jira ticket
is https://issues.apache.org/jira/browse/COLLECTIONS-728


The package has a builder that create ProtoBloomFilters that are then used
to construct real bloom filters based on the desired "shape" of the
Filter.  The question here is should the builder have the build() / with()
methods that is currently has or should the number of methods be pruned?

Currently the builder has the following methods

build()
build(byte)
build(byte[])
build(ByteBuffer)
build(ProtoBloomFilter)
build(String)
with(byte)
with(byte[])
with(ByteBuffer)
with(ProtoBloomFilter)
with(String)

It has been suggested that this should be reduced to

build()
with(byte)
with(byte[])
with(ByteBuffer)
with(ProtoBloomFilter)
with(String)

The reasoning behind this is that is is akin to building a message digest.
The java.security.MessageDigest class has multiple digest() and update()
methods that are logically the same as the build() and with() methods in
the ProtoBloomFilter.Builder.  Thus the ProtoBloomFilter.Builder follows
the same pattern.

If you have an opinion please respond.

Claude
-- 
I like: Like Like - The likeliest place on the web
<http://like-like.xenei.com>
LinkedIn: http://www.linkedin.com/in/claudewarren

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message