ChaiBapchya commented on issue #17597: [CI] Move stacktrace print to console rather than system.err URL: https://github.com/apache/incubator-mxnet/pull/17597#issuecomment-587110492 My point about security was a general point. I just made a note of it here. This was function 'printStackTrace' was already present since #17065 I have only made 1 change (move that logging from std err to std out) - so that it gets print in the console. Not sure what security consideration should I consider? This change of mine doesn't need any change in the existing Script Security (since it has already added printStackTrace to the approved function list) It can be verified here - http://jenkins.mxnet-ci.amazon-ml.com/scriptApproval/ 
---------------------------------------------------------------- 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] With regards, Apache Git Services
