This is an automated email from the ASF dual-hosted git repository.

aherbert pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-collections.git

commit 7b15598da0d4c3da6fff8a9c77ccf4f4f3a7abbd
Author: Alex Herbert <aherb...@apache.org>
AuthorDate: Sun Mar 15 23:26:40 2020 +0000

    Update javadoc for ArrayCountingBloomFilter.
    
    Document that no exception is raised when the filter state transitions
    to invalid.
---
 .../bloomfilter/ArrayCountingBloomFilter.java      | 23 ++++++++++++----------
 1 file changed, 13 insertions(+), 10 deletions(-)

diff --git 
a/src/main/java/org/apache/commons/collections4/bloomfilter/ArrayCountingBloomFilter.java
 
b/src/main/java/org/apache/commons/collections4/bloomfilter/ArrayCountingBloomFilter.java
index 5c30917..3b292e8 100644
--- 
a/src/main/java/org/apache/commons/collections4/bloomfilter/ArrayCountingBloomFilter.java
+++ 
b/src/main/java/org/apache/commons/collections4/bloomfilter/ArrayCountingBloomFilter.java
@@ -29,17 +29,20 @@ import 
org.apache.commons.collections4.bloomfilter.hasher.StaticHasher;
 /**
  * A counting Bloom filter using an array to track counts for each enabled bit
  * index.
- *
- * <p>Any operation that results in negative counts or integer overflow of 
counts will
- * mark this filter as invalid. This transition is not reversible. The counts 
for the
- * filter immediately prior to the operation that create invalid counts can be 
recovered.
- * See the documentation in {@link #isValid()} for details.
- *
- * <p>All the operations in the filter assume the counts are currently valid. 
Behaviour
- * of an invalid filter is undefined. It will no longer function identically 
to a standard
+ * 
+ * <p>Any operation that results in negative counts or integer overflow of
+ * counts will mark this filter as invalid. This transition is not reversible.
+ * The operation is completed in full, no exception is raised and the state is
+ * set to invalid. This allows the counts for the filter immediately prior to 
the
+ * operation that created the invalid state to be recovered. See the 
documentation
+ * in {@link #isValid()} for details.
+ * 
+ * <p>All the operations in the filter assume the counts are currently valid,
+ * for example cardinality or contains operations. Behaviour of an invalid
+ * filter is undefined. It will no longer function identically to a standard
  * Bloom filter that is the merge of all the Bloom filters that have been added
  * to and not later subtracted from the counting Bloom filter.
- *
+ * 
  * <p>The maximum supported number of items that can be stored in the filter is
  * limited by the maximum array size combined with the {@link Shape}. For
  * example an implementation using a {@link Shape} with a false-positive
@@ -47,8 +50,8 @@ import 
org.apache.commons.collections4.bloomfilter.hasher.StaticHasher;
  * approximately 75 million items using 20 hash functions per item with a 
memory
  * consumption of approximately 8 GB.
  *
- * @since 4.5
  * @see Shape
+ * @since 4.5
  */
 public class ArrayCountingBloomFilter extends AbstractBloomFilter implements 
CountingBloomFilter {
 

Reply via email to