mboehm7 commented on pull request #1154:
URL: https://github.com/apache/systemds/pull/1154#issuecomment-770288532


   LGTM - thanks for the extension @tobiasrieger. I only made minor changes 
during the merge: (1) reverted the cumulative time measurement in Timing and 
changed the setup measurement accordingly, (2) added the data partitioning 
timing (time was measured but not collected), (3) reverted the changed test 
logging level to INFO, and (4) some minor formatting changes to avoid huge 
indentation.   


----------------------------------------------------------------
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]


Reply via email to