mboehm7 commented on PR #2052: URL: https://github.com/apache/systemds/pull/2052#issuecomment-2248461550
Thanks for the further improvement @gghsu. May I ask you to resolve the following three issues? First, please test all group-by method like we do in the raJoin tests. Second, please remove the unnecessary print(toString) debug outputs which are overhead hazards with large data. Third, keep the 2-space indentation in dml scripts (which I fixed during previous merges). Thanks, and once these minor issues are resolved, we can merge it it. -- 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
