mboehm7 commented on pull request #1476: URL: https://github.com/apache/systemds/pull/1476#issuecomment-1001774861
LGTM - thanks the both of you. During the merge, I only slightly changed some formatting, and the logic in `updateETFed` in order to make it specific to properties of individual inputs (instead of someFederatedInput and somePrefetch), which gives us more flexibility if some input is known to be broadcast. Furthermore, I added a TODO for refactoring as we might want to generalize and reuse the codegen memo table here too. -- 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]
