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