GitHub user zhaohm3 opened a pull request:

    https://github.com/apache/hive/pull/28

    Fix Hive ParseException while parsing Grouping Sets

    Currently, when Hive parses GROUPING SETS clauses, Hive will throw 
ParseException while parsing the following HQLs:
    
        drop table test;
        create table test(tc1 int, tc2 int, tc3 int);
        
        explain select test.tc1, test.tc2 from test group by test.tc1, test.tc2 
grouping sets(test.tc1, (test.tc1, test.tc2));
        explain select tc1+tc2, tc2 from test group by tc1+tc2, tc2 grouping 
sets(tc2, (tc1 + tc2, tc2));
        
        drop table test;
    
    The following contents show some ParseExctption stacktrace:
    
        2015-01-07 09:53:34,718 INFO [main]: log.PerfLogger 
(PerfLogger.java:PerfLogBegin(108)) - <PERFLOG method=Driver.run 
from=org.apache.hadoop.hive.ql.Driver>
        2015-01-07 09:53:34,719 INFO [main]: log.PerfLogger 
(PerfLogger.java:PerfLogBegin(108)) - <PERFLOG method=TimeToSubmit 
from=org.apache.hadoop.hive.ql.Driver>
        2015-01-07 09:53:34,721 INFO [main]: ql.Driver 
(Driver.java:checkConcurrency(158)) - Concurrency mode is disabled, not 
creating a lock manager
        2015-01-07 09:53:34,721 INFO [main]: log.PerfLogger 
(PerfLogger.java:PerfLogBegin(108)) - <PERFLOG method=compile 
from=org.apache.hadoop.hive.ql.Driver>
        2015-01-07 09:53:34,724 INFO [main]: log.PerfLogger 
(PerfLogger.java:PerfLogBegin(108)) - <PERFLOG method=parse 
from=org.apache.hadoop.hive.ql.Driver>
        2015-01-07 09:53:34,724 INFO [main]: parse.ParseDriver 
(ParseDriver.java:parse(185)) - Parsing command: explain select test.tc1, 
test.tc2 from test group by test.tc1, test.tc2 grouping sets(test.tc1, 
(test.tc1, test.tc2))
        2015-01-07 09:53:34,734 ERROR [main]: ql.Driver 
(SessionState.java:printError(545)) - FAILED: ParseException line 1:105 missing 
) at ',' near '<EOF>'
        line 1:116 extraneous input ')' expecting EOF near '<EOF>'
        org.apache.hadoop.hive.ql.parse.ParseException: line 1:105 missing ) at 
',' near '<EOF>'
        line 1:116 extraneous input ')' expecting EOF near '<EOF>'
        at 
org.apache.hadoop.hive.ql.parse.ParseDriver.parse(ParseDriver.java:210)
        at 
org.apache.hadoop.hive.ql.parse.ParseDriver.parse(ParseDriver.java:166)
        at org.apache.hadoop.hive.ql.Driver.compile(Driver.java:404)
        at org.apache.hadoop.hive.ql.Driver.compile(Driver.java:322)
        at org.apache.hadoop.hive.ql.Driver.compileInternal(Driver.java:975)
        at org.apache.hadoop.hive.ql.Driver.runInternal(Driver.java:1040)
        at org.apache.hadoop.hive.ql.Driver.run(Driver.java:911)
        at org.apache.hadoop.hive.ql.Driver.run(Driver.java:901)
        at 
org.apache.hadoop.hive.cli.CliDriver.processLocalCmd(CliDriver.java:268)
        at org.apache.hadoop.hive.cli.CliDriver.processCmd(CliDriver.java:220)
        at org.apache.hadoop.hive.cli.CliDriver.processLine(CliDriver.java:423)
        at 
org.apache.hadoop.hive.cli.CliDriver.executeDriver(CliDriver.java:792)
        at org.apache.hadoop.hive.cli.CliDriver.run(CliDriver.java:686)
        at org.apache.hadoop.hive.cli.CliDriver.main(CliDriver.java:625)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at org.apache.hadoop.util.RunJar.main(RunJar.java:208)
        2015-01-07 09:53:34,745 INFO [main]: log.PerfLogger 
(PerfLogger.java:PerfLogEnd(135)) - </PERFLOG method=compile 
start=1420595614721 end=1420595614745 duration=24 
from=org.apache.hadoop.hive.ql.Driver>
        2015-01-07 09:53:34,745 INFO [main]: log.PerfLogger 
(PerfLogger.java:PerfLogBegin(108)) - <PERFLOG method=releaseLocks 
from=org.apache.hadoop.hive.ql.Driver>
        2015-01-07 09:53:34,746 INFO [main]: log.PerfLogger 
(PerfLogger.java:PerfLogEnd(135)) - </PERFLOG method=releaseLocks 
start=1420595614745 end=1420595614746 duration=1 
from=org.apache.hadoop.hive.ql.Driver>
        2015-01-07 09:53:34,746 INFO [main]: log.PerfLogger 
(PerfLogger.java:PerfLogBegin(108)) - <PERFLOG method=releaseLocks 
from=org.apache.hadoop.hive.ql.Driver>
        2015-01-07 09:53:34,746 INFO [main]: log.PerfLogger 
(PerfLogger.java:PerfLogEnd(135)) - </PERFLOG method=releaseLocks 
start=1420595614746 end=1420595614746 duration=0 
from=org.apache.hadoop.hive.ql.Driver>
    
    But, Hive will not throw ParseException while handling the follwing HQLs:
    
        drop table test;
        create table test(tc1 int, tc2 int, tc3 int);
        
        explain select tc1, test.tc2 from test group by tc1, test.tc2 grouping 
sets(tc1, (tc1, test.tc2));
        explain select tc1+tc2, tc1 from test group by tc1+tc2, tc1 grouping 
sets(tc1, (tc1, tc1 + tc2));
        explain select test.tc1, test.tc1 + test.tc2 from test group by 
test.tc1, test.tc1 + test.tc2 grouping sets(test.tc1, (test.tc1), (test.tc1 + 
test.tc2));
        
        drop table test;
    
    For more details, see here: https://www.zybuluo.com/Spongcer/note/61369

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/zhaohm3/hive dev

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/hive/pull/28.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #28
    
----
commit fc8a2d4ae3b45c0adc384bb908a41489e02ea560
Author: zhaohm3 <zhao...@asiainfo.com>
Date:   2015-01-05T06:05:00Z

    Modefided  IdentifiersParser.g

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to