Michael Smith has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/21530


Change subject: IMPALA-13166: Speed up getFullyQualifiedSlotPath
......................................................................

IMPALA-13166: Speed up getFullyQualifiedSlotPath

Caches getFullyQualifiedSlotPath for repeat calls, and uses
Arrays.asList to avoid copying primitive arrays.

Implements equals and hashCode on Path to provide a cleaner interface
for lookups on Path.

Speeds up PlannerTest#testManyExpressionPerformance from 21s to 18s in
local testing.

Change-Id: If02d3b20a980508deafd2109a10676ab01e50981
---
M fe/src/main/java/org/apache/impala/analysis/Analyzer.java
M fe/src/main/java/org/apache/impala/analysis/CollectionTableRef.java
M fe/src/main/java/org/apache/impala/analysis/CreateFunctionStmtBase.java
M fe/src/main/java/org/apache/impala/analysis/Path.java
M fe/src/test/java/org/apache/impala/analysis/AnalyzerTest.java
5 files changed, 76 insertions(+), 63 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/30/21530/1
--
To view, visit http://gerrit.cloudera.org:8080/21530
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: If02d3b20a980508deafd2109a10676ab01e50981
Gerrit-Change-Number: 21530
Gerrit-PatchSet: 1
Gerrit-Owner: Michael Smith <[email protected]>

Reply via email to