Re: Review Request 16900: Add "select database()" command to show the current database

2014-02-04 Thread Navis Ryu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16900/ --- (Updated Feb. 5, 2014, 2:13 a.m.) Review request for hive. Changes --- s

Re: Review Request 16900: Add "select database()" command to show the current database

2014-02-02 Thread Navis Ryu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16900/ --- (Updated Feb. 3, 2014, 2:15 a.m.) Review request for hive. Changes --- A

Re: Review Request 16900: Add "select database()" command to show the current database

2014-02-02 Thread Navis Ryu
> On Jan. 15, 2014, 6:15 p.m., Prasad Mujumdar wrote: > > ql/src/java/org/apache/hadoop/hive/ql/exec/mr/ExecDriver.java, line 567 > > > > > > Should this be defined via HiveConf ? Also would be better to add this > >

Re: Review Request 16900: Add "select database()" command to show the current database

2014-01-15 Thread Prasad Mujumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16900/#review31885 --- Very useful functionality! Thanks for patch. Looks fine, a couple of

Review Request 16900: Add "select database()" command to show the current database

2014-01-14 Thread Navis Ryu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16900/ --- Review request for hive. Bugs: HIVE-4144 https://issues.apache.org/jira/bro