jit

Baunsgaard · Oct 21, 2024 · 75972ac · 75972ac
1 parent ccba531
commit 75972ac
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/main/java/org/apache/sysds/runtime/frame/data/compress/CompressedFrameBlockFactory.java b/src/main/java/org/apache/sysds/runtime/frame/data/compress/CompressedFrameBlockFactory.java
@@ -228,11 +228,7 @@ private void logStatistics() {
 				if(in.getColumn(i) instanceof ACompressedArray)
 					sb.append(String.format("Col: %3d, %s\n", i, "Column is already compressed"));
 				else
-				//  if(stats[i].shouldCompress)
 					sb.append(String.format("Col: %3d, %s\n", i, stats[i]));
-				// else
-					// sb.append(String.format("Col: %3d, No Compress, Type: %s\n", //
-						// i, in.getColumn(i).getClass().getSimpleName()));
 			}
 			LOG.debug(sb);
 		}

diff --git a/src/main/java/org/apache/sysds/runtime/transform/encode/CompressedEncode.java b/src/main/java/org/apache/sysds/runtime/transform/encode/CompressedEncode.java
@@ -450,10 +450,14 @@ private AMapToData createMappingAMapToData(Array<?> a, Map<?, Long> map, boolean
 	private static AMapToData createMappingAMapToDataNoNull(Array<?> a, Map<?, Long> map, int si,  AMapToData m, int start, int end) {
 		// TODO push down to underlying array if critical performance to allow JIT compilation.
 		for(int i = start; i < end; i++)
-			m.set(i, map.get(a.get(i)).intValue() - 1);
+			setM(a, map, m, i);
 		return m;
 	}
 
+	private static void setM(Array<?> a, Map<?, Long> map, AMapToData m, int i) {
+		m.set(i, map.get(a.get(i)).intValue() - 1);
+	}
+
 	private static AMapToData createMappingAMapToDataWithNull(Array<?> a, Map<?, Long> map, int si,  AMapToData m, int start, int end) {
 		// TODO push down to underlying array if critical performance to allow JIT compilation.
 		for(int i = start; i < end; i++)