Removed one exclusion as unnecessary in FindBugsExcludeFilter.xml

Documented in the code where the SpotBugs
finds issues that we still suppress. (only 3 places.)

Author: leerho

src/main/java/org/apache/datasketches/kll/KllItemsSketch.java

@@ -243,7 +243,7 @@ public double[] getRanks(final T[] quantiles, final QuantileSearchCriteria searc
   public final KllItemsSketchSortedView<T> getSortedView() {
     if (isEmpty()) { throw new SketchesArgumentException(EMPTY_MSG); }
     refreshSortedView();
-    return kllItemsSV;
+    return kllItemsSV; //SpotBugs EI_EXPOSE_REP, Suppressed by FindBugsExcludeFilter
   }
 
   @Override

src/main/java/org/apache/datasketches/quantilescommon/GenericSortedViewIterator.java

@@ -35,8 +35,8 @@ public class GenericSortedViewIterator<T> implements SortedViewIterator {
   private int index;
 
   public GenericSortedViewIterator(final T[] quantiles, final long[] cumWeights) {
-    this.quantiles = quantiles;
-    this.cumWeights = cumWeights;
+    this.quantiles = quantiles;   //SpotBugs EI_EXPOSE_REP2 suppressed by FindBugsExcludeFilter
+    this.cumWeights = cumWeights; //SpotBugs EI_EXPOSE_REP2 suppressed by FindBugsExcludeFilter
     this.totalN = (cumWeights.length > 0) ? cumWeights[cumWeights.length - 1] : 0;
     index = -1;
   }

src/test/java/org/apache/datasketches/req/ReqDebugImpl.java

@@ -53,7 +53,7 @@ public ReqDebugImpl(final int debugLevel, final String fmt) {
   @Override
   public void emitStart(final ReqSketch sk) {
     if (debugLevel == 0) { return; }
-    this.sk = sk;
+    this.sk = sk; //SpotBugs EI_EXPOSE_REP2 suppressed by FindBugsExcludeFilter
     println("START");
   }
 

tools/FindBugsExcludeFilter.xml

@@ -55,13 +55,9 @@ under the License.
     <Class name="org.apache.datasketches.kll.KllItemsSketch"/>
   </Match>
 
-  <Match>
-    <Bug pattern="EI_EXPOSE_REP2"/>
-    <Class name="org.apache.datasketches.kll.KllItemsSketchSortedView"/>
-  </Match>
-  
   <Match>
     <Bug pattern="EI_EXPOSE_REP2"/>
     <Class name="org.apache.datasketches.quantilescommon.GenericSortedViewIterator"/>
   </Match>
+
 </FindBugsFilter>