Fix and/or filter some SpotBugs

apache · Jan 15, 2020 · f14def3 · f14def3
1 parent 2b8bb61
commit f14def3
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 9 deletions.
diff --git a/src/main/java/org/apache/datasketches/hll/TgtHllType.java b/src/main/java/org/apache/datasketches/hll/TgtHllType.java
@@ -54,7 +54,7 @@
 @SuppressWarnings("javadoc")
 public enum TgtHllType { HLL_4, HLL_6, HLL_8;
 
-  public static final TgtHllType values[] = values();
+  private static final TgtHllType values[] = values();
 
   public static final TgtHllType fromOrdinal(final int typeId) {
     return values[typeId];

diff --git a/src/test/java/org/apache/datasketches/hll/IsomorphicTest.java b/src/test/java/org/apache/datasketches/hll/IsomorphicTest.java
@@ -36,9 +36,6 @@
 @SuppressWarnings({"javadoc", "unused"})
 public class IsomorphicTest {
   long v = 0;
-  final int maxLgK = 12;
-  HllSketch source;
-  Union union;
 
   @Test
   //Merges a type1 to an empty union (heap, HLL_8), and gets result as type1, checks binary equivalence
@@ -227,15 +224,15 @@ public void isomorphicHllMerges2() {
       bytesOut6 = u.getResult(HLL_8).toUpdatableByteArray();//should be identical except for HllAccum
 
       comb = "LgK: " + lgK + ", SkType: HLL_6, Compared with SkType HLL_8";
-      checkArrays(bytesOut8, bytesOut6, comb, true);
+      checkArrays(bytesOut8, bytesOut6, comb, false);
 
       u = buildHeapUnionHllMode(lgK, 0);
       sk = buildHeapSketchHllMode(lgK, HLL_4, 1 << lgK);
       u.update(sk);
       bytesOut4 = u.getResult(HLL_8).toUpdatableByteArray();//should be identical except for HllAccum
 
       comb = "LgK: " + lgK + ", SkType: HLL_4, Compared with SkType HLL_8";
-      checkArrays(bytesOut8, bytesOut4, comb, true);
+      checkArrays(bytesOut8, bytesOut4, comb, false);
     }
   }
 

diff --git a/src/test/java/org/apache/datasketches/hll/UnionCaseTest.java b/src/test/java/org/apache/datasketches/hll/UnionCaseTest.java
@@ -40,11 +40,12 @@
  */
 @SuppressWarnings("javadoc")
 public class UnionCaseTest {
+  private static final String LS = System.getProperty("line.separator");
   long v = 0;
-  final int maxLgK = 12;
+  final static int maxLgK = 12;
   HllSketch source;
-  Union union;
-  String hfmt = "%10s%10s%10s%10s%10s%10s%10s%10s%10s%10s%10s\n";
+  //Union union;
+  String hfmt = "%10s%10s%10s%10s%10s%10s%10s%10s%10s%10s%10s" + LS;
   String hdr = String.format(hfmt, "caseNum","srcLgKStr","gdtLgKStr","srcType","gdtType",
       "srcMem","gdtMem","srcMode","gdtMode","srcOoof","gdtOoof");
 

diff --git a/tools/FindBugsExcludeFilter.xml b/tools/FindBugsExcludeFilter.xml
@@ -44,6 +44,20 @@ under the License.
     <Class name="~.*\.*Test" />
   </Match>
 
+  <Match>
+    <Bug pattern="EQ_COMPARETO_USE_OBJECT_EQUALS" />
+    <Class name="~.*\.Group" />
+  </Match>
+
+  <Match>
+    <Bug pattern="VO_VOLATILE_INCREMENT" />
+    <Class name="~.*\.Concurrent.+QuickSelectSketch" />
+  </Match>
+
+  <Match> 
+    <Class name="~.*\.TestAllTest" />
+  </Match>
+
   <Match>   <!-- Exclude for test classes; too many False Positives. -->
     <Bug pattern="RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT" />
     <Class name="~.*\.*Test" />