[ 
https://issues.apache.org/jira/browse/HIVE-1953?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13087975#comment-13087975
 ] 

jirapos...@reviews.apache.org commented on HIVE-1953:
-----------------------------------------------------


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


Currently if you type something like

EXPLAIN SELECT "abcd
--
efgh" as field from invites;

the string "abcd\n--\nefgh"with the newlines is accepted--this diff removes the 
line with the "--" which seems incorrect.

- Charles


On 2011-08-19 08:10:32, Syed Albiz wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/1587/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2011-08-19 08:10:32)
bq.  
bq.  
bq.  Review request for hive and Yongqiang He.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  skip comments in cliDriver.main() instead of passing down to command 
processors.
bq.  
bq.  
bq.  This addresses bug HIVE-1953.
bq.      https://issues.apache.org/jira/browse/HIVE-1953
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    cli/src/java/org/apache/hadoop/hive/cli/CliDriver.java a2976b5 
bq.  
bq.  Diff: https://reviews.apache.org/r/1587/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  tested build, since the test suite goes through driver, adding a query 
file test case did not work.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Syed
bq.  
bq.



> Hive should process comments in CliDriver
> -----------------------------------------
>
>                 Key: HIVE-1953
>                 URL: https://issues.apache.org/jira/browse/HIVE-1953
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: He Yongqiang
>            Assignee: Syed S. Albiz
>         Attachments: HIVE-1953.1.patch
>
>
> If you put commend before a set command, it will faile. 
> Like this:
> -- TestSerDe is a user defined serde where the default delimiter is Ctrl-B
> -- the user is overwriting it with ctrlC
> set hive.input.format=org.apache.hadoop.hive.ql.io.HiveInputFormat;
> Hive should process the comment in CliDriver, and ignore the comment right 
> away, instead of passing it to the downstream processors.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to