marcoabreu commented on issue #17808: [WIP] Windows dev environment 
configuration, update install instructions from source in the docs
URL: https://github.com/apache/incubator-mxnet/pull/17808#issuecomment-608055780
 
 
   Connectivity errors were never an issue with Windows slaves. It sounds like 
you've changed something as part of generating the AMIs. Thus, I'm hesitant to 
just accept a bunch of more changes (which have not been made transparent) as a 
result to fix a previously created issue.
   
   Please just revert stuff back to what it was and stop putting duckttape over 
things. The build used to work and the slaves used to be stable. But I have 
only seen attempts to change things but no rootcausing (I know about the 2GB 
limit). Some action resulted in our build breaking and the only sane choice is 
to revert whatever was done and then do these efforts again.
   
   I will not accept multiple layers of fixes which could have been avoided if 
the changes were properly developed and tested in a separate environment. At 
the moment, it seems like open heart surgery on a productive system. 

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

Reply via email to