You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2020/10/27 21:06:45 UTC

[GitHub] [incubator-mxnet] ptrendx commented on pull request #19432: [WIP] Revert "Remove cleanup on side threads (#19378)"

ptrendx commented on pull request #19432:
URL: https://github.com/apache/incubator-mxnet/pull/19432#issuecomment-717539522


   @sxjscience Sure, I understand that there is an issue somewhere and I'm happy to help resolve it. That said, neither this PR nor the linked issue explains why the failure happens with that change enabled (I assume here that you verified that a commit just before #19378 passes). If this was about reverting a feature it would be fine, but here we are reverting a bugfix (or bug workaround to be more precise), so I think it makes sense to make sure we understand what is going on here before we reintroduce a problem.


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