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/06/02 16:28:28 UTC

[GitHub] [systemds] phaniarnab commented on pull request #1286: [SYSTEMDS-2889] Add initial lineage debugger for CP instructions

phaniarnab commented on pull request #1286:
URL: https://github.com/apache/systemds/pull/1286#issuecomment-853172304


   Thanks for the patch @bnyra. Before merging, I moved the debugging-related maintenance from Instruction to CPInstruction as calling acquireRead can be problematic for federated and GPU instructions, and removed serialization of the debug byte if debugger is disabled.


-- 
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:
users@infra.apache.org