ywcb00 opened a new pull request #1390: URL: https://github.com/apache/systemds/pull/1390
Hi, This PR contains a small fix for the MV case to keep the output federated. The problem was that the branch to create a federated output was only taken if it was forced federated. I have now changed it so that the partial output is only set if it is forced federated, and the federated output is set if it is not forced local. The other changes inside AggregateBinaryFEDInstruction should not change the functional behaviour of the instruction. I only tried to cleanup some code redundancies that could be merged. This PR also includes an implicit change to the FederatedTokenizeTest, where I removed the creation of the fourth thread because it is never used. Thanks for review :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
