Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11180 )
Change subject: IMPALA-7420: Different error messages for internal cancellation ...................................................................... Patch Set 2: Code-Review-1 Talked offline with bikram, I think this might need some rethought - e.g. are we sure the error messages don't leak out, should we add some logging when they do, should put in the effort now to use a separate error code to enable dchecks etc. -- To view, visit http://gerrit.cloudera.org:8080/11180 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Icfd69adfa4b06a68ebef9c350af66f18e0309b8d Gerrit-Change-Number: 11180 Gerrit-PatchSet: 2 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Pooja Nilangekar <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Sat, 11 Aug 2018 00:01:59 +0000 Gerrit-HasComments: No
