hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Harish Butani" <>
Subject Re: Review Request 15525: HIVE-3107: Improve semantic analyzer to better handle column name references in group by/sort by clauses
Date Sat, 16 Nov 2013 02:34:41 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Nov. 16, 2013, 2:34 a.m.)

Review request for hive, Ashutosh Chauhan and Xuefu Zhang.


- changed altInvRslvMap to contain one entry
- carry forward alt mapping for Select
- added test case from HIVE-1922

Bugs: hive-3107

Repository: hive-git


Following queries all fail with various SemanticExceptions:
explain select t.c from t group by c;
explain select t.c from t group by c sort by t.c; 
explain select t.c as c0 from t group by c0;
explain select t.c from t group by t.c sort by t.c; 

Diffs (updated)

  ql/src/java/org/apache/hadoop/hive/ql/parse/ 7a7f3ef 
  ql/src/java/org/apache/hadoop/hive/ql/parse/ 908546e 
  ql/src/java/org/apache/hadoop/hive/ql/parse/ d797407 
  ql/src/test/queries/clientnegative/notable_alias3.q 6cc3e87 
  ql/src/test/queries/clientpositive/groupby_resolution.q PRE-CREATION 
  ql/src/test/queries/clientpositive/notable_alias3.q PRE-CREATION 
  ql/src/test/results/clientnegative/notable_alias3.q.out cadca6e 
  ql/src/test/results/clientpositive/groupby_resolution.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/notable_alias3.q.out PRE-CREATION 
  ql/src/test/results/compiler/errors/nonkey_groupby.q.out a13d45d 
  ql/src/test/results/compiler/plan/groupby1.q.xml 485c323 
  ql/src/test/results/compiler/plan/groupby5.q.xml abdbff0 



added groupby_resolution.q that tests the jira listed.
Also added tests for having and windowing.


Harish Butani

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message