You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hive.apache.org by Rajesh Balamohan <ra...@gmail.com> on 2017/05/25 07:21:10 UTC

Review Request 59569: HIVE-15300: Reuse table information in SemanticAnalyzer::getMetaData to reduce compilation time

-----------------------------------------------------------
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