impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Behm (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-4985: use parquet stats of nested types for dynamic pruning
Date Tue, 21 Nov 2017 23:57:17 GMT
Alex Behm has posted comments on this change. ( )

Change subject: IMPALA-4985: use parquet stats of nested types for dynamic pruning

Patch Set 6:


lgtm after final nits
File fe/src/main/java/org/apache/impala/analysis/
PS6, Line 154:    * Checks if slotRef refers to an array "pos" pseudo-column.
Checks if this SlotRef
PS6, Line 160:   public boolean isArrayPosReference() {

(we typically abbreviate Reference with Ref)
File fe/src/main/java/org/apache/impala/planner/
PS5, Line 439:     Preconditions.checkState(slotRef.getDesc().isScanSlot());
> ah, didn't understand this as "move this method to the SlotRef class". Done
Sorry! I realize my comment was not clear. Thanks for addressing.
File fe/src/main/java/org/apache/impala/planner/
PS6, Line 541:       // It is assumed that analysis adds these guards such that they are correct,
guards -> filters
File testdata/workloads/functional-planner/queries/PlannerTest/parquet-filtering.test:
PS5, Line 96: ---- PLAN
> that's the lowest level (at the moment) that prints out "parquet statistics
Got it. Might want to change that, but definitely not in this patch.

The motivation for changing is that dealing with conflicts in planner tests is becoming increasingly

To view, visit
To unsubscribe, visit

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0c99e20cb080b504442cd5376ea3e046016158fe
Gerrit-Change-Number: 8480
Gerrit-PatchSet: 6
Gerrit-Owner: Vuk Ercegovac <>
Gerrit-Reviewer: Alex Behm <>
Gerrit-Reviewer: Lars Volker <>
Gerrit-Reviewer: Vuk Ercegovac <>
Gerrit-Comment-Date: Tue, 21 Nov 2017 23:57:17 +0000
Gerrit-HasComments: Yes

  • Unnamed multipart/alternative (inline, 8-Bit, 0 bytes)
View raw message