Sailesh Mukil has posted comments on this change. Change subject: IMPALA-5331: Use new libHDFS API to address "Unknown Error 255" ......................................................................
Patch Set 7: (2 comments) http://gerrit.cloudera.org:8080/#/c/6894/6/tests/data_errors/test_data_errors.py File tests/data_errors/test_data_errors.py: Line 78: try: > I wonder about stacking all of these asserts in one block. If any of them f Yes, that makes sense. If it fails, we'd want to know why. I've added a line that prints the stderr if it's not empty. This should have sufficient information to be able to debug if any of these commands fail. We'd want the safe mode to be turned off regardless of success or failure, since we don't want the effect of this test to leak into other tests if anything gets run after this. Line 97: # Confirm that we were able to get the root cause. > If this is the happy path, this is fine. If the assert fails though, you mi I've added a message that says "Couldn't turn Safe mode OFF". -- To view, visit http://gerrit.cloudera.org:8080/6894 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I181e316ed63b70b94d4f7a7557d398a931bb171d Gerrit-PatchSet: 7 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com> Gerrit-Reviewer: David Knupp <dkn...@cloudera.com> Gerrit-Reviewer: Henry Robinson <he...@cloudera.com> Gerrit-Reviewer: Matthew Jacobs <m...@cloudera.com> Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com> Gerrit-HasComments: Yes
