You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by ha...@apache.org on 2013/10/16 17:32:45 UTC
svn commit: r1532800 -
/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcInputFormat.java
Author: hashutosh
Date: Wed Oct 16 15:32:45 2013
New Revision: 1532800
URL: http://svn.apache.org/r1532800
Log:
HIVE-5546 : A change in ORCInputFormat made by HIVE4113 was reverted by HIVE5391 (Yin Huai via Ashutosh Chauhan)
Modified:
hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcInputFormat.java
Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcInputFormat.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcInputFormat.java?rev=1532800&r1=1532799&r2=1532800&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcInputFormat.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcInputFormat.java Wed Oct 16 15:32:45 2013
@@ -200,10 +200,8 @@ public class OrcInputFormat implements
* @return true for each column that should be included
*/
public static boolean[] findIncludedColumns(List<OrcProto.Type> types, Configuration conf) {
- String includedStr = conf.get(ColumnProjectionUtils.READ_COLUMN_IDS_CONF_STR);
- LOG.info("included column ids = " + includedStr);
- if (ColumnProjectionUtils.isReadAllColumns(conf) ||
- includedStr == null || includedStr.trim().length() == 0) {
+ LOG.info("included column ids = " + conf.get(ColumnProjectionUtils.READ_COLUMN_IDS_CONF_STR));
+ if (ColumnProjectionUtils.isReadAllColumns(conf)) {
return null;
} else {
int numColumns = types.size();