hadoop-common-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Prasad Chakka (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HADOOP-4501) [Hive] CLI not supporting multiline queries from an input file
Date Mon, 03 Nov 2008 21:50:44 GMT

    [ https://issues.apache.org/jira/browse/HADOOP-4501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12644818#action_12644818

Prasad Chakka commented on HADOOP-4501:

Hi Jeremy,

Couple of issues
1) Should line 166 be in inside the if (m.matches()) condition?
2) line 185: i think if loop gets executed even if the last line doesn't end with a ';'
3) I think processLine should be used instead of processCmd. Otherwise the code can't handle
cmds like 'select a.* from tabA a; select \nb.* from tabB b'
4) also, could you update the patch with new code available for 0.17. There have been some
changes that make the commands case insensitive.

It will be good if you can make similar changes to code that handles interactive commands
(in main()). If not we can do that in a separate JIRA as well. 

> [Hive] CLI not supporting multiline queries from an input file
> --------------------------------------------------------------
>                 Key: HADOOP-4501
>                 URL: https://issues.apache.org/jira/browse/HADOOP-4501
>             Project: Hadoop Core
>          Issue Type: Improvement
>          Components: contrib/hive
>    Affects Versions: 0.17.3
>            Reporter: Jeremy Huylebroeck
>            Priority: Trivial
>             Fix For: 0.17.3
>         Attachments: patch-0.17.txt, patch-trunk.txt, patch2-0.17.txt
> calling "hive -f filename" allows to pass HQL queries from a file.
> but the queries can't be on multiple lines as we would expect by reading Hive documentation.
> a proposed fix is to modify CliDriver.processReader.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message