hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Carl Steinbach" <c...@cloudera.com>
Subject Re: Review Request: HIVE-3100: Integrate SQLLine command line editor
Date Mon, 18 Jun 2012 07:56:01 GMT

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



LICENSE
<https://reviews.apache.org/r/5381/#comment17928>

    This is the BSD license template, not the actual license. Looks like the updated BSD license
is available in the SQLLine CVS repository here: http://sqlline.cvs.sourceforge.net/viewvc/sqlline/LICENSE?view=markup
    



NOTICE
<https://reviews.apache.org/r/5381/#comment17929>

    Please change this to:
    
    This product includes/uses SQLLine (http://sqlline.sourceforge.net),
    Copyright (c) 2002, 2003, 2004, 2005 Marc Prud'hommeaux <mwp1@cornell.edu>.



bin/ext/sqlline.sh
<https://reviews.apache.org/r/5381/#comment17933>

    If no hostname is specified this script should probably  run the driver in embedded mode
(e.g. with a connect string equal to "jdbc:hive://", or see TestJdbcDriver for another example).



bin/ext/sqlline.sh
<https://reviews.apache.org/r/5381/#comment17932>

    The default HiveServer listen port is 10000.



bin/ext/sqlline.sh
<https://reviews.apache.org/r/5381/#comment17934>

    "-n foo -p bar"?
    
     Is it possible to modify this in order to make it possible for the user to pass options
to SQLLine from the command line (e.g. --showWarnings, --outputformat, etc)?
    
    Why is "--silent" hardcoded here (and is this --silent=true or --silent=false)?



bin/ext/sqlline.sh
<https://reviews.apache.org/r/5381/#comment17943>

    I see this error message three times when I start the sqlline CLI: "Error: Method not
supported (state=,code=0)"
    
    Any idea what's causing that?
    



bin/ext/sqlline.sh
<https://reviews.apache.org/r/5381/#comment17936>

    This prints out the SQLLine options, but I don't think those options are actually accessible
if you start it using this script.



bin/ext/util/execHiveCmd.sh
<https://reviews.apache.org/r/5381/#comment17944>

    I don't think there's any benefit to putting this function here. Please move this back
to sqlline.sh
    



bin/ext/util/execHiveCmd.sh
<https://reviews.apache.org/r/5381/#comment17945>

    We removed support for Hadoop versions < 20 several versions ago. Please remove this
comment.



bin/ext/util/execHiveCmd.sh
<https://reviews.apache.org/r/5381/#comment17946>

    I dumped the values of these variables and they don't look right:
    
    CLASS = "-u"
    HIVE_OPTS = ""
    @ = "jdbc:hive://localhost:10000/default -n foo -p bar -d org.apache.hadoop.hive.jdbc.HiveDriver
--silent"
    
    Including $HIVE_OPTS isn't appropriate in this circumstance. That environment variable
is supposed to contain a list of "-hiveconf xx" settings, and the logic for processing these
options is located in o.a.h.hive.cli.OptionsProcessor, which doesn't get invoked when you
start SQLLine like this.
    
    One option for fixing this is to create a BeeLine java wrapper around SQLLine, and invoke
SQLLine's main() method from BeeLine.main() after processing the -hiveconf options.



ivy.xml
<https://reviews.apache.org/r/5381/#comment17930>

    Why is this listed as a dependency both here and in cli/ivy.xml? I think this instance
can be removed.



ivy/libraries.properties
<https://reviews.apache.org/r/5381/#comment17931>

    We should probably depend on version 1.0.2 assuming it works (http://sourceforge.net/projects/sqlline/files/sqlline/1.0.2/).


- Carl Steinbach


On June 18, 2012, 5:59 a.m., Prasad Mujumdar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/5381/
> -----------------------------------------------------------
> 
> (Updated June 18, 2012, 5:59 a.m.)
> 
> 
> Review request for hive and Carl Steinbach.
> 
> 
> Description
> -------
> 
> The patch is to include command line SQL editor SQLLine into Hive distribution. The tool
can be invoked using 'hive --service sqlline <host> [port]'. It requires the HiveServer
running on the given host/port.
> 
> The ivy dependencies are updated to include sqlline. The hive scripts are updated to
executing SQLLine with the required connection URL and other command line options. The LICENSE
and NOTICE files are updated to include SQLLine information.
> 
> 
> Diffs
> -----
> 
>   LICENSE 05085da 
>   NOTICE 871fdde 
>   bin/ext/sqlline.sh PRE-CREATION 
>   bin/ext/util/execHiveCmd.sh 167cc40 
>   cli/ivy.xml ab949b1 
>   ivy.xml e83437e 
>   ivy/ivysettings.xml fb6f4b8 
>   ivy/libraries.properties 8461da1 
> 
> Diff: https://reviews.apache.org/r/5381/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Prasad Mujumdar
> 
>


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