Baunsgaard commented on pull request #1110: URL: https://github.com/apache/systemds/pull/1110#issuecomment-730042985
> adding ternary aggregates is fine, but has nothing to do with the title of disabling rmvar right? Regarding the deleted wait on cleanup, please be aware that this might require federated worker changes as now they have to deal with concurrent RPC on a single execution context (not just concurrent RPC on different execution contexts that are managed in a concurrent hashmap). So it has a chance of brittle failures when these RPCs conflict on symbol table updates. no ternary has nothing to do with the pr.. but it was one of the operation s making L2SVM slow, so i just added it. Unfortunally it is only relevant if y is federated as well. But in that case, the wide network becomes very slow. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
