----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41931/#review112988 -----------------------------------------------------------
Ship it! ql/src/test/org/apache/hadoop/hive/ql/parse/TestParseDriver.java (line 101) <https://reviews.apache.org/r/41931/#comment173467> Please avoid System.out. - Amareshwari Sriramadasu On Jan. 5, 2016, 3:10 p.m., Rajat Khandelwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41931/ > ----------------------------------------------------------- > > (Updated Jan. 5, 2016, 3:10 p.m.) > > > Review request for hive. > > > Bugs: HIVE-12776 > https://issues.apache.org/jira/browse/HIVE-12776 > > > Repository: hive-git > > > Description > ------- > > Extensions that use Hive QL as their standard language, will benefit from > this. > > Apache Lens uses HQL as its language of choice. To support that, it depends > on a fork of Hive, which has such code. I'm planning to port that to Apache > Hive. > > Relevant commit: > https://github.com/InMobi/hive/commit/7caea9ed1d269c1cd1d1326cb39c1db7e0bf2bba#diff-fb3acd67881ceb02e83c2e42cf70beef > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/parse/ParseDriver.java > 90a9a4e6bbefdb4679bc76e21661ba4658a1df3c > ql/src/test/org/apache/hadoop/hive/ql/parse/TestParseDriver.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/41931/diff/ > > > Testing > ------- > > > Thanks, > > Rajat Khandelwal > >
