[hive] branch master updated: HIVE-22816 : QueryCache: Queries using views can have them cached after CTE expansion (Gopal V via Jesus Camacho Rodriguez)

[hashutosh]

This is an automated email from the ASF dual-hosted git repository.

hashutosh pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git


The following commit(s) were added to refs/heads/master by this push:
     new f826283  HIVE-22816 : QueryCache: Queries using views can have them 
cached after CTE expansion (Gopal V via Jesus Camacho Rodriguez)
f826283 is described below

commit f826283d36503eb7598b1e33b10bd5d7d1ddd074
Author: Gopal V <gop...@apache.org>
AuthorDate: Thu Feb 20 20:01:01 2020 -0800

    HIVE-22816 : QueryCache: Queries using views can have them cached after CTE 
expansion (Gopal V via Jesus Camacho Rodriguez)
    
    Signed-off-by: Ashutosh Chauhan <hashut...@apache.org>
---
 ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 
b/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
index f46739e..2328eed 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
@@ -15279,8 +15279,10 @@ public class SemanticAnalyzer extends 
BaseSemanticAnalyzer {
   }
 
   private List<Table> getNonTransactionalTables() {
+    // views have been expanded by CBO already and can be ignored
     return tablesFromReadEntities(inputs)
         .stream()
+        .filter(table -> !table.isView())
         .filter(table -> !AcidUtils.isTransactionalTable(table))
         .collect(Collectors.toList());
   }