----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25956/#review54342 -----------------------------------------------------------
data/conf/hive-site.xml <https://reviews.apache.org/r/25956/#comment94466> Why are we turning on CBO? ql/src/java/org/apache/hadoop/hive/ql/optimizer/optiq/translator/ASTConverter.java <https://reviews.apache.org/r/25956/#comment94467> This needs to get table alias if it was alaised. Hive-8237 introduced a method on ReloptHiveTable to get the table alias (which will use either user provided alias or will fall back to table name) ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java <https://reviews.apache.org/r/25956/#comment94468> This collides with HIVE-8237. We now keep track of full TableName (DB.tblName, Table Alias). - John Pullokkaran On Sept. 23, 2014, 9:41 p.m., Ashutosh Chauhan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25956/ > ----------------------------------------------------------- > > (Updated Sept. 23, 2014, 9:41 p.m.) > > > Review request for hive, John Pullokkaran and Harish Butani. > > > Bugs: HIVE-8238 > https://issues.apache.org/jira/browse/HIVE-8238 > > > Repository: hive-git > > > Description > ------- > > [CBO] Preserve subquery alias while generating ast > > > Diffs > ----- > > data/conf/hive-site.xml ed3ee42 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/optiq/translator/ASTConverter.java > 77427e8 > ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 943cc5c > > Diff: https://reviews.apache.org/r/25956/diff/ > > > Testing > ------- > > authorization_view_sqlstd.q > > > Thanks, > > Ashutosh Chauhan > >
