impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Behm (Code Review)" <>
Subject [Impala-CR](cdh5-trunk) IMPALA-2805: Order conjuncts based on selectivity and cost
Date Thu, 21 Apr 2016 22:51:27 GMT
Alex Behm has posted comments on this change.

Change subject: IMPALA-2805: Order conjuncts based on selectivity and cost

Patch Set 14:

File fe/src/main/java/com/cloudera/impala/analysis/

Line 1254:     SlotRef ref = e.unwrapSlotRef(false);
CHAR types have a fixed size

Line 1256:       if (ref.getResolvedPath() != null && ref.getResolvedPath().destColumn()
!= null &&
shorter way to get the stats ref.getDesc().getStats()
File fe/src/main/java/com/cloudera/impala/planner/

Line 69:   public void init(Analyzer analyzer) throws ImpalaException {
exchange cannot have conjuncts
File fe/src/main/java/com/cloudera/impala/planner/

Line 165:     conjuncts_ = orderConjunctsByCost(conjuncts_);
this node has no conjuncts, see function comment
File testdata/workloads/functional-planner/queries/PlannerTest/conjunct-ordering.test:

Line 1: # We don't estimate selectivity for 'slot_ref = slot_ref'
Marcel, what is your take on the testing here? I think we should either expand the tests to
cover conjuncts in more places, or we should leave them out and let the existing tests be
the coverage. What do you think?

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I02279a26fbc6308ac5eb819d78345fc010469034
Gerrit-PatchSet: 14
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Thomas Tauber-Marshall <>
Gerrit-Reviewer: Alex Behm <>
Gerrit-Reviewer: Henry Robinson <>
Gerrit-Reviewer: Marcel Kornacker <>
Gerrit-Reviewer: Matthew Jacobs <>
Gerrit-Reviewer: Mostafa Mokhtar <>
Gerrit-Reviewer: Thomas Tauber-Marshall <>
Gerrit-HasComments: Yes

View raw message