-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15965/#review29657
-----------------------------------------------------------



ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g
<https://reviews.apache.org/r/15965/#comment57068>

    Please remove the trailing space


Patch looks good. However, please fill out the fields of the review request, 
such as bug number branch. The topic should  start with the bug number also. 
Please find other review request as an example.

- Xuefu Zhang


On Dec. 3, 2013, 3:14 p.m., Yu Zhao wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15965/
> -----------------------------------------------------------
> 
> (Updated Dec. 3, 2013, 3:14 p.m.)
> 
> 
> Review request for hive.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> {code}
> hive (default)> show partitions log.history;
> FAILED: ParseException line 1:19 missing EOF at '.' near 'log'
> {code}
> 
> This command doesn't work when we use db.tablename as tablename in its 
> parameter.
> 
> 
> Diffs
> -----
> 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g 2343a2c 
>   ql/src/test/queries/clientpositive/show_partitions.q 7fa7b82 
>   ql/src/test/results/clientpositive/show_partitions.q.out 0b0a97a 
>   ql/src/test/results/clientpositive/showparts.q.out fccd131 
> 
> Diff: https://reviews.apache.org/r/15965/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Yu Zhao
> 
>

Reply via email to