mboehm7 commented on PR #2044: URL: https://github.com/apache/systemds/pull/2044#issuecomment-2211838568
LGTM - thanks for the patch @gghsu. During the merge, I added the default join method, renamed the new method to 'sort-merge', and let a few TODOs for future improvements (in a follow up project). In the future, please start a new PR instead of including the old commits (which creates unnecessary work for conflict resolution). Thanks. -- 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-unsubscr...@systemds.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org