impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Attila Jeges (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-3548: Prune runtime filters based on query options in the FE
Date Thu, 03 Aug 2017 18:13:40 GMT
Attila Jeges has posted comments on this change.

Change subject: IMPALA-3548: Prune runtime filters based on query options in the FE

Patch Set 2:


Thanks for the review!
File fe/src/main/java/org/apache/impala/planner/

PS1, Line 119: RuntimeFilterGener
> unnecessary

PS1, Line 121: 
> single line?
File fe/src/main/java/org/apache/impala/planner/

Line 538:    * The assigned filters are the ones for which 'scanNode' can be used a destination
> update this comment

PS1, Line 549: Preconditions.checkNotNull(scanNode);
             :     Analyzer analyzer = ctx.getRootAnalyzer();
> single line?

PS1, Line 590: e
> nit: space before ':'
File testdata/workloads/functional-query/queries/QueryTest/runtime_filters_distrib_pruning.test:

Line 1: ====
> I suppose these are written as query tests and not planner tests because th
Correct. These test cases should be in the planner test suite, but implementing them this
way is more straightforward.

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: Id0f0b200e02442edcad8df3979f652d66c6e52eb
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Attila Jeges <>
Gerrit-Reviewer: Alex Behm <>
Gerrit-Reviewer: Attila Jeges <>
Gerrit-Reviewer: Dimitris Tsirogiannis <>
Gerrit-Reviewer: Matthew Jacobs <>
Gerrit-Reviewer: Michael Ho <>
Gerrit-Reviewer: Thomas Tauber-Marshall <>
Gerrit-HasComments: Yes

View raw message