Michael Smith has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/21533 )

Change subject: IMPALA-13166: Optimize ExprSubstitutionMap allocation
......................................................................

IMPALA-13166: Optimize ExprSubstitutionMap allocation

Allocates HashMap and Lists with known sizes to avoid resizing them
during initial construction. HashMap uses default capacity of 16 and
load factor of .75, and we increase initial capacity as needed.

Has no measurable impact on performance, but may help with garbage
collection.

Change-Id: If5e56401508f30e3066b1a3758d69d89a288adcd
Reviewed-on: http://gerrit.cloudera.org:8080/21533
Reviewed-by: Michael Smith <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M fe/src/main/java/org/apache/impala/analysis/ExprSubstitutionMap.java
1 file changed, 19 insertions(+), 5 deletions(-)

Approvals:
  Michael Smith: Looks good to me, approved
  Impala Public Jenkins: Verified

--
To view, visit http://gerrit.cloudera.org:8080/21533
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: If5e56401508f30e3066b1a3758d69d89a288adcd
Gerrit-Change-Number: 21533
Gerrit-PatchSet: 8
Gerrit-Owner: Michael Smith <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Peter Rozsa <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>

Reply via email to