You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by ke...@apache.org on 2013/03/22 18:16:21 UTC

svn commit: r1459905 - in /hive/trunk/ql/src: java/org/apache/hadoop/hive/ql/parse/ColumnAccessAnalyzer.java test/results/clientpositive/column_access_stats.q.out

Author: kevinwilfong
Date: Fri Mar 22 17:16:20 2013
New Revision: 1459905

URL: http://svn.apache.org/r1459905
Log:
HIVE-4092. Store complete names of tables in column access analyzer (Samuel Yuan via kevinwilfong)

Modified:
    hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/ColumnAccessAnalyzer.java
    hive/trunk/ql/src/test/results/clientpositive/column_access_stats.q.out

Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/ColumnAccessAnalyzer.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/ColumnAccessAnalyzer.java?rev=1459905&r1=1459904&r2=1459905&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/ColumnAccessAnalyzer.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/ColumnAccessAnalyzer.java Fri Mar 22 17:16:20 2013
@@ -43,7 +43,7 @@ public class ColumnAccessAnalyzer {
     Map<TableScanOperator, Table> topOps = pGraphContext.getTopToTable();
     for (TableScanOperator op : topOps.keySet()) {
       Table table = topOps.get(op);
-      String tableName = table.getTableName();
+      String tableName = table.getCompleteName();
       List<FieldSchema> tableCols = table.getAllCols();
       for (int i : op.getNeededColumnIDs()) {
         columnAccessInfo.add(tableName, tableCols.get(i).getName());

Modified: hive/trunk/ql/src/test/results/clientpositive/column_access_stats.q.out
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/test/results/clientpositive/column_access_stats.q.out?rev=1459905&r1=1459904&r2=1459905&view=diff
==============================================================================
--- hive/trunk/ql/src/test/results/clientpositive/column_access_stats.q.out (original)
+++ hive/trunk/ql/src/test/results/clientpositive/column_access_stats.q.out Fri Mar 22 17:16:20 2013
@@ -14,7 +14,7 @@ SELECT key FROM T1 ORDER BY key
 PREHOOK: type: QUERY
 PREHOOK: Input: default@t1
 #### A masked pattern was here ####
-Table:t1
+Table:default@t1
 Columns:key
 
 1
@@ -27,7 +27,7 @@ PREHOOK: query: SELECT key, val FROM T1 
 PREHOOK: type: QUERY
 PREHOOK: Input: default@t1
 #### A masked pattern was here ####
-Table:t1
+Table:default@t1
 Columns:key,val
 
 1	11
@@ -95,7 +95,7 @@ PREHOOK: query: SELECT key FROM (SELECT 
 PREHOOK: type: QUERY
 PREHOOK: Input: default@t1
 #### A masked pattern was here ####
-Table:t1
+Table:default@t1
 Columns:key
 
 1
@@ -152,7 +152,7 @@ PREHOOK: query: SELECT k FROM (SELECT ke
 PREHOOK: type: QUERY
 PREHOOK: Input: default@t1
 #### A masked pattern was here ####
-Table:t1
+Table:default@t1
 Columns:key
 
 1
@@ -165,7 +165,7 @@ PREHOOK: query: SELECT key + 1 as k FROM
 PREHOOK: type: QUERY
 PREHOOK: Input: default@t1
 #### A masked pattern was here ####
-Table:t1
+Table:default@t1
 Columns:key
 
 2.0
@@ -178,7 +178,7 @@ PREHOOK: query: SELECT key + val as k FR
 PREHOOK: type: QUERY
 PREHOOK: Input: default@t1
 #### A masked pattern was here ####
-Table:t1
+Table:default@t1
 Columns:key,val
 
 12.0
@@ -274,7 +274,7 @@ SELECT val as c FROM T1
 PREHOOK: type: QUERY
 PREHOOK: Input: default@t1
 #### A masked pattern was here ####
-Table:t1
+Table:default@t1
 Columns:key,val
 
 1
@@ -375,7 +375,7 @@ SELECT key as c FROM T1
 PREHOOK: type: QUERY
 PREHOOK: Input: default@t1
 #### A masked pattern was here ####
-Table:t1
+Table:default@t1
 Columns:key
 
 1
@@ -398,7 +398,7 @@ PREHOOK: type: QUERY
 PREHOOK: Input: default@t1
 PREHOOK: Output: default@t2
 PREHOOK: Output: default@t3
-Table:t1
+Table:default@t1
 Columns:key,val
 
 PREHOOK: query: -- Simple joins
@@ -410,10 +410,10 @@ PREHOOK: type: QUERY
 PREHOOK: Input: default@t1
 PREHOOK: Input: default@t2
 #### A masked pattern was here ####
-Table:t2
+Table:default@t2
 Columns:key,val
 
-Table:t1
+Table:default@t1
 Columns:key,val
 
 1	11	1	1
@@ -523,10 +523,10 @@ PREHOOK: type: QUERY
 PREHOOK: Input: default@t1
 PREHOOK: Input: default@t2
 #### A masked pattern was here ####
-Table:t2
+Table:default@t2
 Columns:key
 
-Table:t1
+Table:default@t1
 Columns:key
 
 1
@@ -543,10 +543,10 @@ PREHOOK: type: QUERY
 PREHOOK: Input: default@t1
 PREHOOK: Input: default@t2
 #### A masked pattern was here ####
-Table:t2
+Table:default@t2
 Columns:key,val
 
-Table:t1
+Table:default@t1
 Columns:key,val
 
 PREHOOK: query: -- Map join
@@ -558,10 +558,10 @@ PREHOOK: type: QUERY
 PREHOOK: Input: default@t1
 PREHOOK: Input: default@t2
 #### A masked pattern was here ####
-Table:t2
+Table:default@t2
 Columns:key,val
 
-Table:t1
+Table:default@t1
 Columns:key,val
 
 1	11	1	1
@@ -701,10 +701,10 @@ PREHOOK: type: QUERY
 PREHOOK: Input: default@t1
 PREHOOK: Input: default@t2
 #### A masked pattern was here ####
-Table:t2
+Table:default@t2
 Columns:key,val
 
-Table:t1
+Table:default@t1
 Columns:key,val
 
 PREHOOK: query: EXPLAIN
@@ -840,10 +840,10 @@ PREHOOK: type: QUERY
 PREHOOK: Input: default@t1
 PREHOOK: Input: default@t2
 #### A masked pattern was here ####
-Table:t2
+Table:default@t2
 Columns:key,val
 
-Table:t1
+Table:default@t1
 Columns:key,val
 
 PREHOOK: query: -- Join followed by join
@@ -1050,15 +1050,15 @@ PREHOOK: Input: default@t1
 PREHOOK: Input: default@t2
 PREHOOK: Input: default@t3
 #### A masked pattern was here ####
-Table:t3
-Columns:key,val
-
-Table:t2
+Table:default@t2
 Columns:key
 
-Table:t1
+Table:default@t1
 Columns:key
 
+Table:default@t3
+Columns:key,val
+
 1	1	11.0
 2	2	12.0
 3	3	13.0