Re: Review Request 62782: HIVE-17706 Add a possibility to run the BeeLine tests on the default database

2017-10-06 Thread Peter Vary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62782/ --- (Updated Oct. 6, 2017, 10:43 a.m.) Review request for hive and Barna Zsombor

Re: Review Request 62782: HIVE-17706 Add a possibility to run the BeeLine tests on the default database

2017-10-06 Thread Barna Zsombor Klara
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62782/#review187240 --- Ship it! Ship It! - Barna Zsombor Klara On Oct. 5, 2017,

Re: Review Request 62782: HIVE-17706 Add a possibility to run the BeeLine tests on the default database

2017-10-05 Thread Peter Vary
> On Oct. 5, 2017, 11:58 a.m., Barna Zsombor Klara wrote: > > itests/util/src/main/java/org/apache/hadoop/hive/cli/control/CoreBeeLineDriver.java > > Line 119 (original), 122 (patched) > > > > > > This may be OK,

Re: Review Request 62782: HIVE-17706 Add a possibility to run the BeeLine tests on the default database

2017-10-05 Thread Peter Vary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62782/ --- (Updated Oct. 5, 2017, 4:39 p.m.) Review request for hive and Barna Zsombor

Re: Review Request 62782: HIVE-17706 Add a possibility to run the BeeLine tests on the default database

2017-10-05 Thread Barna Zsombor Klara
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62782/#review187168 --- Thank you for the patch Peter, looking good. I don't have a clear

Review Request 62782: HIVE-17706 Add a possibility to run the BeeLine tests on the default database

2017-10-05 Thread Peter Vary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62782/ --- Review request for hive and Barna Zsombor Klara. Bugs: HIVE-17706