Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8123 )
Change subject: IMPALA-5870: Improve runtime profile for partial sort ...................................................................... Patch Set 3: The GVO failed because there was a kudu insert test that checked the profile for the 'SpilledRuns: 0' line. Since that is now gone, the test checks for 'SortType: Partial' (the real point of the test was that the query wouldn't have run previously due to the mem limit, which still applies, so we're not particularly losing any coverage). -- To view, visit http://gerrit.cloudera.org:8080/8123 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2b15af78d8299db8edc44ff820c85db1cbe0be1b Gerrit-Change-Number: 8123 Gerrit-PatchSet: 3 Gerrit-Owner: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Comment-Date: Tue, 26 Sep 2017 19:46:06 +0000 Gerrit-HasComments: No