Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/8652 )
Change subject: IMPALA-6241: timeout in admission control test under ASAN ...................................................................... Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/8652/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/8652/1//COMMIT_MSG@10 PS1, Line 10: : While testing that fix, I discovered that the ASAN build detection logic : was failing silently Sorry to hear this wasn't working. Thanks for making an improvement. http://gerrit.cloudera.org:8080/#/c/8652/1/infra/python/deps/requirements.txt File infra/python/deps/requirements.txt: http://gerrit.cloudera.org:8080/#/c/8652/1/infra/python/deps/requirements.txt@45 PS1, Line 45: monkeypatch == 0.1rc3 I don't think anything else is using this, so it can be removed. http://gerrit.cloudera.org:8080/#/c/8652/1/tests/common/environ.py File tests/common/environ.py: http://gerrit.cloudera.org:8080/#/c/8652/1/tests/common/environ.py@19 PS1, Line 19: import pytest Unused; remove. http://gerrit.cloudera.org:8080/#/c/8652/1/tests/common/environ.py@112 PS1, Line 112: is_dev Do you want to add ubsan to this method? http://gerrit.cloudera.org:8080/#/c/8652/1/tests/common/environ.py@122 PS1, Line 122: runs_slowly Do you want to add ubsan to this method? -- To view, visit http://gerrit.cloudera.org:8080/8652 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia4ed949cac99b9925f72e19e4adaa2ead370b536 Gerrit-Change-Number: 8652 Gerrit-PatchSet: 1 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Michael Brown <mi...@cloudera.com> Gerrit-Comment-Date: Mon, 27 Nov 2017 23:46:38 +0000 Gerrit-HasComments: Yes