----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58454/#review172617 -----------------------------------------------------------
beeline/src/java/org/apache/hive/beeline/OutputFile.java Lines 49 (patched) <https://reviews.apache.org/r/58454/#comment245700> it seems like this 'coverter' currently leaked into this OutputFile class...which is part of the production code... Wouldn't it be possible to avoid this? I'm think in something like: ``` myBeeLine.setOutputFile(new ConvertedOutputFile(new SortConverterre(), myBeeLine.getOutputFile() ) ) ``` this may possibly also enable to remove the enums/etc from production code and move them into testing only beeline/src/java/org/apache/hive/beeline/OutputFile.java Lines 133 (patched) <https://reviews.apache.org/r/58454/#comment245699> I don't think that 'Supported' should appear in the enums name...it doesn't really helps in describing what this is :) beeline/src/java/org/apache/hive/beeline/OutputFile.java Lines 134 (patched) <https://reviews.apache.org/r/58454/#comment245701> it would be nice to add the processing class to the enum; it would enable the cleanup of some switched ``` SORT_QUERY_RESULTS(SortAndDigestPrintStream.class) ``` - Zoltan Haindrich On April 18, 2017, 8:32 a.m., Peter Vary wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58454/ > ----------------------------------------------------------- > > (Updated April 18, 2017, 8:32 a.m.) > > > Review request for hive, Zoltan Haindrich, Vihang Karajgaonkar, and Yongzhi > Chen. > > > Bugs: HIVE-16449 > https://issues.apache.org/jira/browse/HIVE-16449 > > > Repository: hive-git > > > Description > ------- > > Changes to support the following features in the BeeLine Qtests: > – SORT_QUERY_RESULTS > – HASH_QUERY_RESULTS > – SORT_AND_HASH_QUERY_RESULTS > > The patch contains the following changes: > - Added the possibility to the OutputFile to add the existing FetchConverters > - Indicate the start and the end of the result fetching when fetching the > resultset > - Parsed the query files for indentifing the cases when the converters are > needed > > > Diffs > ----- > > beeline/src/java/org/apache/hive/beeline/Commands.java 08d53ca > beeline/src/java/org/apache/hive/beeline/OutputFile.java 1014af3 > > itests/util/src/main/java/org/apache/hadoop/hive/cli/control/CoreBeeLineDriver.java > 8c7057c > itests/util/src/main/java/org/apache/hive/beeline/QFile.java 0bde529 > itests/util/src/main/java/org/apache/hive/beeline/QFileBeeLineClient.java > f1b53f7 > ql/src/test/results/clientpositive/beeline/smb_mapjoin_1.q.out c943b03 > ql/src/test/results/clientpositive/beeline/smb_mapjoin_2.q.out 1ea6553 > ql/src/test/results/clientpositive/beeline/smb_mapjoin_3.q.out f639ba4 > > > Diff: https://reviews.apache.org/r/58454/diff/4/ > > > Testing > ------- > > Checked the output changes of the existing BeeLineDriver tests > > > Thanks, > > Peter Vary > >
