Anurag Mantripragada has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15921 )

Change subject: IMPALA-9749: ASAN builds should not run FE tests.
......................................................................


Patch Set 4:

(1 comment)

Made the changes suggested by Thomas. I'm started a private-parametrized asan 
build. I will wait for the job to finish.

http://gerrit.cloudera.org:8080/#/c/15921/3/bin/run-all-tests.sh
File bin/run-all-tests.sh:

http://gerrit.cloudera.org:8080/#/c/15921/3/bin/run-all-tests.sh@41
PS3, Line 41: # Parametrized Test Options
            : # Run FE Tests
            : : ${FE_TEST:=true}
            : # Run Backend Tests
            : : ${BE_TEST:=true}
> Yeah of course, this is happening because we previously required both CLUST
As per Thomas's suggestion above, I moved the FE custom cluster tests to the 
branch in FE_TESTS = true. Also added restart script from L166 after the tests.



--
To view, visit http://gerrit.cloudera.org:8080/15921
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I26c469a20032bdc1f4f0bb3938d9f1c50163c99a
Gerrit-Change-Number: 15921
Gerrit-PatchSet: 4
Gerrit-Owner: Anurag Mantripragada <anu...@cloudera.com>
Gerrit-Reviewer: Anurag Mantripragada <anu...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Sahil Takiar <stak...@cloudera.com>
Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com>
Gerrit-Comment-Date: Fri, 22 May 2020 21:43:24 +0000
Gerrit-HasComments: Yes

Reply via email to