JiaLiangC opened a new pull request, #3780: URL: https://github.com/apache/ambari/pull/3780
….Mapping ## What changes were proposed in this pull request? 1.in Python 3, collections.Mapping has been moved. The correct import path is now collections.abc.Mapping. 2.Because all subprocess child processes have universal_newlines=True, all outputs are strings. However, os.read returns bytes, so str_buffer += data_chunk will result in an error. ## How was this patch tested? (Please explain how this patch was tested. Ex: unit tests, manual tests) (If this patch involves UI changes, please attach a screen-shot; otherwise, remove this) Please review [Ambari Contributing Guide](https://cwiki.apache.org/confluence/display/AMBARI/How+to+Contribute) before opening a pull request. -- 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. To unsubscribe, e-mail: dev-unsubscr...@ambari.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@ambari.apache.org For additional commands, e-mail: dev-h...@ambari.apache.org
