Github user rxin commented on a diff in the pull request:

    https://github.com/apache/spark/pull/10883#discussion_r50630042
  
    --- Diff: 
common/sketch/src/main/java/org/apache/spark/util/sketch/BloomFilter.java ---
    @@ -0,0 +1,142 @@
    +/*
    + * Licensed to the Apache Software Foundation (ASF) under one or more
    + * contributor license agreements.  See the NOTICE file distributed with
    + * this work for additional information regarding copyright ownership.
    + * The ASF licenses this file to You under the Apache License, Version 2.0
    + * (the "License"); you may not use this file except in compliance with
    + * the License.  You may obtain a copy of the License at
    + *
    + *    http://www.apache.org/licenses/LICENSE-2.0
    + *
    + * Unless required by applicable law or agreed to in writing, software
    + * distributed under the License is distributed on an "AS IS" BASIS,
    + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    + * See the License for the specific language governing permissions and
    + * limitations under the License.
    + */
    +
    +package org.apache.spark.util.sketch;
    +
    +/**
    + * A Bloom filter is a space-efficient probabilistic data structure, that 
is used to test whether
    + * an element is a member of a set. It returns false when the element is 
definitely not in the
    + * set, returns true when the element is probably in the set.
    + *
    + * Internally a Bloom filter is initialized with 2 information: how many 
space to use(number of
    + * bits) and how many hash values to calculate for each record.  To get as 
lower false positive
    + * probability as possible, user should call {@link BloomFilter#create} to 
automatically pick a
    + * best combination of these 2 parameters.
    + *
    + * Currently the following data types are supported:
    + * <ul>
    + *   <li>{@link Byte}</li>
    + *   <li>{@link Short}</li>
    + *   <li>{@link Integer}</li>
    + *   <li>{@link Long}</li>
    + *   <li>{@link String}</li>
    + * </ul>
    + *
    + * The implementation is largely based on the {@code BloomFilter} class 
from guava.
    + */
    +public abstract class BloomFilter {
    +  /**
    +   * Returns the probability that {@linkplain #mightContain(Object)} will 
erroneously return
    +   * {@code true} for an object that has not actually been put in the 
{@code BloomFilter}.
    +   *
    +   * <p>Ideally, this number should be close to the {@code fpp} parameter
    +   * passed in to create this bloom filter, or smaller. If it is
    +   * significantly higher, it is usually the case that too many elements 
(more than
    +   * expected) have been put in the {@code BloomFilter}, degenerating it.
    +   */
    +  public abstract double expectedFpp();
    +
    +  /**
    +   * Returns the number of bits in the underlying bit array.
    +   */
    +  public abstract long bitSize();
    +
    +  /**
    +   * Puts an element into this {@code BloomFilter}. Ensures that 
subsequent invocations of
    +   * {@link #mightContain(Object)} with the same element will always 
return {@code true}.
    +   *
    +   * @return true if the bloom filter's bits changed as a result of this 
operation. If the bits
    +   *     changed, this is <i>definitely</i> the first time {@code object} 
has been added to the
    +   *     filter. If the bits haven't changed, this <i>might</i> be the 
first time {@code object}
    +   *     has been added to the filter. Note that {@code put(t)} always 
returns the
    +   *     <i>opposite</i> result to what {@code mightContain(t)} would have 
returned at the time
    +   *     it is called."
    --- End diff --
    
    an extra trailing "


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to