You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by rb...@apache.org on 2016/12/19 09:59:36 UTC

hive git commit: HIVE-15463: EncodedReaderImpl::readEncodedColumns in io elevator path printing lots of logs (Rajesh Balamohan reviewed by Prasanth Jayachandran)

Repository: hive
Updated Branches:
  refs/heads/master c6a7edd5a -> 5790e6ad3


HIVE-15463: EncodedReaderImpl::readEncodedColumns in io elevator path printing lots of logs (Rajesh Balamohan reviewed by Prasanth Jayachandran)


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/5790e6ad
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/5790e6ad
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/5790e6ad

Branch: refs/heads/master
Commit: 5790e6ad3d3a645e7a15cfccfdaba8146eddd6c3
Parents: c6a7edd
Author: Rajesh Balamohan <rb...@apache.org>
Authored: Mon Dec 19 15:29:27 2016 +0530
Committer: Rajesh Balamohan <rb...@apache.org>
Committed: Mon Dec 19 15:29:27 2016 +0530

----------------------------------------------------------------------
 .../apache/hadoop/hive/ql/io/orc/encoded/EncodedReaderImpl.java | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/5790e6ad/ql/src/java/org/apache/hadoop/hive/ql/io/orc/encoded/EncodedReaderImpl.java
----------------------------------------------------------------------
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/io/orc/encoded/EncodedReaderImpl.java b/ql/src/java/org/apache/hadoop/hive/ql/io/orc/encoded/EncodedReaderImpl.java
index 8f0c237..0c5b675 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/io/orc/encoded/EncodedReaderImpl.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/io/orc/encoded/EncodedReaderImpl.java
@@ -207,7 +207,6 @@ class EncodedReaderImpl implements EncodedReader {
       OrcProto.RowIndex[] indexes, List<OrcProto.ColumnEncoding> encodings, List<OrcProto.Stream> streamList,
       boolean[] included, boolean[][] colRgs,
       Consumer<OrcEncodedColumnBatch> consumer) throws IOException {
-    isTracingEnabled = true;
     // Note: for now we don't have to setError here, caller will setError if we throw.
     // We are also not supposed to call setDone, since we are only part of the operation.
     long stripeOffset = stripe.getOffset();
@@ -216,7 +215,7 @@ class EncodedReaderImpl implements EncodedReader {
     // 1.1. Figure out which columns have a present stream
     boolean[] hasNull = RecordReaderUtils.findPresentStreamsByColumn(streamList, types);
     if (isTracingEnabled) {
-      LOG.error("The following columns have PRESENT streams: " + arrayToString(hasNull));
+      LOG.trace("The following columns have PRESENT streams: " + arrayToString(hasNull));
     }
 
     // We assume stream list is sorted by column and that non-data
@@ -229,7 +228,7 @@ class EncodedReaderImpl implements EncodedReader {
       if (!included[i]) continue;
       colCtxs[i] = new ColumnReadContext(i, encodings.get(i), indexes[i], ++colRgIx);
       if (isTracingEnabled) {
-        LOG.error("Creating context: " + colCtxs[i].toString());
+        LOG.trace("Creating context: " + colCtxs[i].toString());
       }
     }
     boolean isCompressed = (codec != null);