hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Rajesh Balamohan <rajesh.balamo...@gmail.com>
Subject Review Request 59569: HIVE-15300: Reuse table information in SemanticAnalyzer::getMetaData to reduce compilation time
Date Thu, 25 May 2017 07:21:10 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59569/
-----------------------------------------------------------

Review request for hive, Ashutosh Chauhan, Jesús Camacho Rodríguez, and Pengcheng Xu.


Repository: hive-git


Description
-------

Queries like Q14, Q88 etc do a lot of table lookups in SemanticAnalyzer for the same set of
tables. Every lookup costs couple of DB calls. We already have "tabNameToTabObject" which
caches table name to Table. This patch attempts to reduce these lookup calls by using information
available in the cache. It makes a deep copy after getting the item from cache, so that even
any modification downstream does not cause issues.


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 43a61ce 


Diff: https://reviews.apache.org/r/59569/diff/1/


Testing
-------

Jenkins passed. TestCliDriver.testCliDriver[index_serde] failure is not related to this patch,
it passed in local environment.


Thanks,

Rajesh Balamohan


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message