[SYSTEMDS-3729] Fix federate response of roll function

min-guk · mboehm7 · commit 63b99e5efb15 · 2024-10-24T13:10:12.000+02:00
This patch removes unnecessary data transfer and data sharing, which was there for debugging only. Closes #2131.
diff --git a/src/main/java/org/apache/sysds/runtime/instructions/fed/ReorgFEDInstruction.java b/src/main/java/org/apache/sysds/runtime/instructions/fed/ReorgFEDInstruction.java
@@ -429,7 +429,7 @@ public FederatedResponse execute(ExecutionContext ec, Data... data) {
 				resBlock = oriBlock;
 			}
 			ec.setMatrixOutput(String.valueOf(_outputID), resBlock);
-			return new FederatedResponse(FederatedResponse.ResponseType.SUCCESS, resBlock);
+			return new FederatedResponse(FederatedResponse.ResponseType.SUCCESS);
 		}
 
 		@Override

Original file line number	Diff line number	Diff line change
`@@ -429,7 +429,7 @@ public FederatedResponse execute(ExecutionContext ec, Data... data) {`
`429`	`429`	`resBlock = oriBlock;`
`430`	`430`	`}`
`431`	`431`	`ec.setMatrixOutput(String.valueOf(_outputID), resBlock);`
`432`		`- return new FederatedResponse(FederatedResponse.ResponseType.SUCCESS, resBlock);`
	`432`	`+ return new FederatedResponse(FederatedResponse.ResponseType.SUCCESS);`
`433`	`433`	`}`
`434`	`434`
`435`	`435`	`@Override`