Skip to content

Commit ef49168

Browse files
committed
Bugfix fix
1 parent 8fca8e6 commit ef49168

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

src/main/java/org/apache/sysds/hops/rewriter/RewriterRuleCollection.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1675,9 +1675,7 @@ public static void canonicalExpandAfterFlattening(final List<RewriterRule> rules
16751675
.as(UUID.randomUUID().toString())
16761676
.withInstruction("sum")
16771677
.withOps(newIdxExpr);
1678-
System.out.println("Copying index list: " + newIdxExpr.toParsableString(ctx));
16791678
RewriterUtils.copyIndexList(newIdxExpr);
1680-
System.out.println("After copy: " + newIdxExpr.toParsableString(ctx));
16811679
newIdxExpr.refreshReturnType(ctx);
16821680
newSum.consolidate(ctx);
16831681
newArgList.getOperands().add(newSum);

src/main/java/org/apache/sysds/hops/rewriter/utils/RewriterUtils.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -869,7 +869,8 @@ public static void copyIndexList(RewriterStatement idxExprRoot) {
869869
return null;
870870
});
871871

872-
idxExprRoot.getOperands().set(1, out);
872+
if (out != null)
873+
idxExprRoot.getOperands().set(1, out);
873874
}
874875

875876
public static void retargetIndexExpressions(RewriterStatement rootExpr, UUID oldIdxId, RewriterStatement newStatement) {

0 commit comments

Comments
 (0)