Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/10155 )
Change subject: IMPALA-6905: support regexes with more verifiers ...................................................................... Patch Set 1: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/10155/1/testdata/workloads/functional-query/queries/QueryTest/stats-extrapolation.test File testdata/workloads/functional-query/queries/QueryTest/stats-extrapolation.test: http://gerrit.cloudera.org:8080/#/c/10155/1/testdata/workloads/functional-query/queries/QueryTest/stats-extrapolation.test@17 PS1, Line 17: row_regex:.* extrapolated-rows=unavailable.* > Yes, I didn't see the point in updating mem-estimate here every time we cha Thanks, I was missing that context. In the future we might want to group all RM stuff together similar to "stored statistics", but for now this seems ok. -- To view, visit http://gerrit.cloudera.org:8080/10155 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia7a28d421c8e7cd84b14d07fcb71b76449156409 Gerrit-Change-Number: 10155 Gerrit-PatchSet: 1 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Wed, 25 Apr 2018 20:10:40 +0000 Gerrit-HasComments: Yes
