----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13383/#review33280 -----------------------------------------------------------
service/src/java/org/apache/hive/service/cli/session/HiveSessionImpl.java <https://reviews.apache.org/r/13383/#comment62690> Do you think we should do this at HiveServer2 startup rather than at session level? - Vaibhav Gumashta On Jan. 31, 2014, 1:29 a.m., Thejas Nair wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13383/ > ----------------------------------------------------------- > > (Updated Jan. 31, 2014, 1:29 a.m.) > > > Review request for hive. > > > Bugs: HIVE-4545 > https://issues.apache.org/jira/browse/HIVE-4545 > > > Repository: hive-git > > > Description > ------- > > HIVE-3140 changed behavior of 'DESCRIBE table;' to be like 'DESCRIBE > FORMATTED table;'. HIVE-3140 introduced changes to not print header in > 'DESCRIBE table;'. But jdbc/odbc calls still get fields padded with space for > the 'DESCRIBE table;' query. > > As the jdbc/odbc results are not for direct human consumption the space > padding should not be done for hive server2. > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 22bb22d > itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcDriver2.java > 74fd53f > ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java a926f1e > > ql/src/java/org/apache/hadoop/hive/ql/metadata/formatting/JsonMetaDataFormatter.java > 54b02cb > > ql/src/java/org/apache/hadoop/hive/ql/metadata/formatting/MetaDataFormatUtils.java > de788f7 > > ql/src/java/org/apache/hadoop/hive/ql/metadata/formatting/MetaDataFormatter.java > 4c40034 > > ql/src/java/org/apache/hadoop/hive/ql/metadata/formatting/TextMetaDataFormatter.java > 9b0d482 > service/src/java/org/apache/hive/service/cli/session/HiveSessionImpl.java > 445c858 > > Diff: https://reviews.apache.org/r/13383/diff/ > > > Testing > ------- > > Updated TestJdbcDriver2 unit tests > > > Thanks, > > Thejas Nair > >
