[MINOR] Fix formatting and method name violations

Author: mboehm7

src/main/java/org/apache/sysds/runtime/instructions/ooc/CachingStream.java

@@ -692,6 +692,7 @@ public void setIXTransform(BiFunction<Boolean, IndexRange, IndexRange> transform
 		throw new UnsupportedOperationException();
 	}
 
+	@SuppressWarnings("unchecked")
 	public void setSubscriber(Consumer<OOCStream.QueueCallback<IndexedMatrixValue>> subscriber, boolean incrConsumers) {
 		if(_deletable)
 			throw new DMLRuntimeException("Cannot register a new subscriber on " + this + " because has been flagged for deletion");

src/main/java/org/apache/sysds/runtime/instructions/ooc/OOCInstruction.java

@@ -205,7 +205,7 @@ protected <T> OOCStream<T> mergeOOCStreams(List<OOCStream<T>> streams) {
 		return new MergedOOCStream<>(streams);
 	}
 
-	@SuppressWarnings({"varargs", "unchecked"})
+	@SuppressWarnings({"unchecked"})
 	protected <T> OOCStream<T> mergeOOCStreams(OOCStream<T>... streams) {
 		return new MergedOOCStream<>(streams);
 	}
@@ -1093,7 +1093,6 @@ protected <T> CompletableFuture<Void> submitOOCTasks(final List<OOCStream<T>> qu
 					};
 
 					if(callback instanceof OOCStream.GroupQueueCallback<?>) {
-						@SuppressWarnings("unchecked")
 						OOCStream.GroupQueueCallback<T> group = (OOCStream.GroupQueueCallback<T>) callback;
 
 						if(localFuture.isDone()) {

src/main/java/org/apache/sysds/runtime/ooc/stream/FilteredOOCStream.java

@@ -85,7 +85,6 @@ public void setSubscriber(Consumer<QueueCallback<T>> subscriber) {
 			}
 
 			if(cb instanceof OOCStream.GroupQueueCallback<?>) {
-				@SuppressWarnings("unchecked")
 				OOCStream.GroupQueueCallback<T> group = (OOCStream.GroupQueueCallback<T>) cb;
 				for(int i = 0; i < group.size(); i++) {
 					QueueCallback<T> sub = group.getCallback(i);

src/main/java/org/apache/sysds/runtime/ooc/stream/SplittingOOCStream.java

@@ -35,6 +35,7 @@ public class SplittingOOCStream<T> implements OOCStream<T> {
 	private OOCStream<T> _sourceStream;
 	private SubOOCStream<T>[] _subStreams;
 
+	@SuppressWarnings("unchecked")
 	public SplittingOOCStream(OOCStream<T> sourceStream, Function<T, Integer> partitionFunc, int numPartitions) {
 		_sourceStream = sourceStream;
 		_subStreams = new SubOOCStream[numPartitions];
@@ -75,7 +76,6 @@ public SplittingOOCStream(OOCStream<T> sourceStream, Function<T, Integer> partit
 					}
 
 					if(cb instanceof OOCStream.GroupQueueCallback<?>) {
-						@SuppressWarnings("unchecked")
 						OOCStream.GroupQueueCallback<T> group = (OOCStream.GroupQueueCallback<T>) cb;
 						for(int gi = 0; gi < group.size(); gi++) {
 							OOCStream.QueueCallback<T> sub = group.getCallback(gi);

src/test/java/org/apache/sysds/test/component/matrixmult/MatrixMultTransposedPerformanceTest.java

@@ -31,21 +31,21 @@ public class MatrixMultTransposedPerformanceTest {
 	private final int k = 200;
 
 	@Test
-	public void testPerf_1_NoTransA_TransB() {
+	public void testPerf1NoTransATransB() {
 		System.out.println("Case: C = A %*% t(B)");
 		runTest(false, true);
 		System.out.println();
 	}
 
 	@Test
-	public void testPerf_2_TransA_NoTransB() {
+	public void testPerf2TransANoTransB() {
 		System.out.println("Case: C = t(A) %*% B");
 		runTest(true, false);
 		System.out.println();
 	}
 
 	@Test
-	public void testPerf_3_TransA_TransB() {
+	public void testPerf3TransATransB() {
 		System.out.println("Case: C = t(A) %*% t(B)");
 		runTest(true, true);
 	}

src/test/java/org/apache/sysds/test/component/matrixmult/MatrixMultTransposedTest.java

@@ -30,62 +30,62 @@
 
 public class MatrixMultTransposedTest {
 
-    // run multiple random scenarios
-    @Test
-    public void testCase_noTransA_TransB() {
-        for(int i=0; i<10; i++) {
-            runTest(false, true);
-        }
-    }
-
-    @Test
-    public void testCase_TransA_NoTransB() {
-        for(int i=0; i<10; i++) {
-            runTest(true, false);
-        }
-    }
-
-    @Test
-    public void testCase_TransA_TransB() {
-        for(int i=0; i<10; i++) {
-            runTest(true, true);
-        }
-    }
-
-    private void runTest(boolean tA, boolean tB) {
-        Random rand = new Random();
-
-        // generate random dimensions between 1 and 300
-        int m = rand.nextInt(300) + 1;
-        int n = rand.nextInt(300) + 1;
-        int k = rand.nextInt(300) + 1;
-
-
-        int rowsA = tA ? k : m;
-        int colsA = tA ? m : k;
-        int rowsB = tB ? n : k;
-        int colsB = tB ? k : n;
-
-        MatrixBlock ma = MatrixBlock.randOperations(rowsA, colsA, 1.0, -1, 1, "uniform", 7);
-        MatrixBlock mb = MatrixBlock.randOperations(rowsB, colsB, 1.0, -1, 1, "uniform", 3);
-
-        MatrixBlock mc = new MatrixBlock(m, n, false);
-        mc.allocateDenseBlock();
-
-        DenseBlock a = ma.getDenseBlock();
-        DenseBlock b = mb.getDenseBlock();
-        DenseBlock c = mc.getDenseBlock();
-
-        LibMatrixMult.matrixMultDenseDenseMM(a, b, c, tA, tB, n, k, 0, m, 0, n);
-
-        mc.recomputeNonZeros();
-
-        // calc true result with existing methods
-        MatrixBlock ma_in = tA ? LibMatrixReorg.transpose(ma) : ma;
-        MatrixBlock mb_in = tB ? LibMatrixReorg.transpose(mb) : mb;
-        MatrixBlock expected = LibMatrixMult.matrixMult(ma_in, mb_in);
-
-        // compare results
-        TestUtils.compareMatrices(expected, mc, 1e-8);
-    }
+	// run multiple random scenarios
+	@Test
+	public void testCaseNoTransATransB() {
+		for(int i=0; i<10; i++) {
+			runTest(false, true);
+		}
+	}
+
+	@Test
+	public void testCaseTransANoTransB() {
+		for(int i=0; i<10; i++) {
+			runTest(true, false);
+		}
+	}
+
+	@Test
+	public void testCaseTransATransB() {
+		for(int i=0; i<10; i++) {
+			runTest(true, true);
+		}
+	}
+
+	private void runTest(boolean tA, boolean tB) {
+		Random rand = new Random();
+
+		// generate random dimensions between 1 and 300
+		int m = rand.nextInt(300) + 1;
+		int n = rand.nextInt(300) + 1;
+		int k = rand.nextInt(300) + 1;
+
+
+		int rowsA = tA ? k : m;
+		int colsA = tA ? m : k;
+		int rowsB = tB ? n : k;
+		int colsB = tB ? k : n;
+
+		MatrixBlock ma = MatrixBlock.randOperations(rowsA, colsA, 1.0, -1, 1, "uniform", 7);
+		MatrixBlock mb = MatrixBlock.randOperations(rowsB, colsB, 1.0, -1, 1, "uniform", 3);
+
+		MatrixBlock mc = new MatrixBlock(m, n, false);
+		mc.allocateDenseBlock();
+
+		DenseBlock a = ma.getDenseBlock();
+		DenseBlock b = mb.getDenseBlock();
+		DenseBlock c = mc.getDenseBlock();
+
+		LibMatrixMult.matrixMultDenseDenseMM(a, b, c, tA, tB, n, k, 0, m, 0, n);
+
+		mc.recomputeNonZeros();
+
+		// calc true result with existing methods
+		MatrixBlock ma_in = tA ? LibMatrixReorg.transpose(ma) : ma;
+		MatrixBlock mb_in = tB ? LibMatrixReorg.transpose(mb) : mb;
+		MatrixBlock expected = LibMatrixMult.matrixMult(ma_in, mb_in);
+
+		// compare results
+		TestUtils.compareMatrices(expected, mc, 1e-8);
+	}
 }