> On July 26, 2014, 12:34 a.m., Szehon Ho wrote: > > data/scripts/q_test_init.sql, line 8 > > <https://reviews.apache.org/r/23953/diff/1/?file=642522#file642522line8> > > > > Is there a lot of work to implement using hiveconf variable like > > BeelineDriver did? If not, can we consider it?
Turns out we can do it. Thanks for telling that variable substitutions are possible with hive cli as well. Updated patch. - Ashish ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23953/#review48791 ----------------------------------------------------------- On July 28, 2014, 11:16 p.m., Ashish Singh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23953/ > ----------------------------------------------------------- > > (Updated July 28, 2014, 11:16 p.m.) > > > Review request for hive. > > > Bugs: HIVE-7519 > https://issues.apache.org/jira/browse/HIVE-7519 > > > Repository: hive-git > > > Description > ------- > > HIVE-7519: Refactor QTestUtil to remove its duplication with QFileClient for > qtest setup and teardown > > > Diffs > ----- > > data/scripts/q_test_cleanup.sql 31bd7205d85916ea352f715f2fd1462efc788208 > data/scripts/q_test_init.sql 12afdf391132e3fdd219aaa581e1f2e210d6dee2 > itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java > 2fefa067791bd74412c0b4efb697dc0d8bb03cd7 > > Diff: https://reviews.apache.org/r/23953/diff/ > > > Testing > ------- > > qTests. > > > Thanks, > > Ashish Singh > >
