Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10722
Change subject: IMPALA-7174: fix test_cancellation for RELEASE builds ...................................................................... IMPALA-7174: fix test_cancellation for RELEASE builds The test was DOA when run against a release build because the debug actions that it depends on were disabled. The fix is to enable the debug actions for release builds, similar to other debug actions. I assume the original motivation of the NDEBUG checks was to avoid adding overhead to release builds. The cost is minimised by quickly checking whether the string is empty before proceeding with any further work. Testing: Looped the test on a release build. Change-Id: I41da7b5ac58a468a8ed117777969906f63df6d4b --- M be/src/scheduling/admission-controller.cc M be/src/service/client-request-state.cc M be/src/util/debug-util.cc M be/src/util/debug-util.h 4 files changed, 12 insertions(+), 17 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/22/10722/1 -- To view, visit http://gerrit.cloudera.org:8080/10722 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I41da7b5ac58a468a8ed117777969906f63df6d4b Gerrit-Change-Number: 10722 Gerrit-PatchSet: 1 Gerrit-Owner: Tim Armstrong <[email protected]>
