You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@systemds.apache.org by GitBox <gi...@apache.org> on 2021/08/18 15:14:13 UTC
[GitHub] [systemds] sebwrede opened a new pull request #1367: [SYSTEMDS-3018] Basic Federated Planner
sebwrede opened a new pull request #1367:
URL: https://github.com/apache/systemds/pull/1367
This PR contains various additions necessary for building a federated planner, which will generate and select execution plans for federated executions. The cost estimation currently added is not connected to the decisions made by the federated planner. This will be done in later PRs, where the data structure for describing the relation between Hops with different FOUT/LOUT/NONE values and the corresponding costs will be implemented.
--
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: dev-unsubscribe@systemds.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [systemds] mboehm7 commented on pull request #1367: [SYSTEMDS-3018] Basic Federated Planner
Posted by GitBox <gi...@apache.org>.
mboehm7 commented on pull request #1367:
URL: https://github.com/apache/systemds/pull/1367#issuecomment-917132984
LGTM - thanks for the patch @sebwrede. This cost analyzer and initial rewriter is a good start toward a full enumeration framework for federated execution plans.
--
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: dev-unsubscribe@systemds.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [systemds] sebwrede commented on pull request #1367: [SYSTEMDS-3018] Basic Federated Planner
Posted by GitBox <gi...@apache.org>.
sebwrede commented on pull request #1367:
URL: https://github.com/apache/systemds/pull/1367#issuecomment-910277584
The federated cost estimation has been moved to a separate class and tests of the cost estimation has been added.
This PR is ready 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: dev-unsubscribe@systemds.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [systemds] mboehm7 edited a comment on pull request #1367: [SYSTEMDS-3018] Basic Federated Planner
Posted by GitBox <gi...@apache.org>.
mboehm7 edited a comment on pull request #1367:
URL: https://github.com/apache/systemds/pull/1367#issuecomment-917132984
LGTM - thanks for the patch @sebwrede. This cost analyzer and initial rewriter is a good start toward a full enumeration framework for federated execution plans. During the merge, I fixed a few warnings, formatting, and code duplication (e.g., reading dml scripts in the tests).
--
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: dev-unsubscribe@systemds.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [systemds] asfgit closed pull request #1367: [SYSTEMDS-3018] Basic Federated Planner
Posted by GitBox <gi...@apache.org>.
asfgit closed pull request #1367:
URL: https://github.com/apache/systemds/pull/1367
--
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: dev-unsubscribe@systemds.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org