You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@orc.apache.org by do...@apache.org on 2021/09/04 06:40:35 UTC

[orc] branch main updated: ORC-983: Revise filter processing log level/location/message (#896)

This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/orc.git


The following commit(s) were added to refs/heads/main by this push:
     new 7cfa787  ORC-983: Revise filter processing log level/location/message (#896)
7cfa787 is described below

commit 7cfa787f60487dd7dcc45f0eeaea63b5f56f0360
Author: Pavan Lanka <pl...@apple.com>
AuthorDate: Fri Sep 3 23:40:28 2021 -0700

    ORC-983: Revise filter processing log level/location/message (#896)
    
    ### What changes were proposed in this pull request?
    
    Couple of the log statements related to filter processing have been lowered from INFO to DEBUG level.
    
    ### Why are the changes needed?
    
    Make the logging less verbose.
    
    ### How was this patch tested?
    
    Regression testing as there is no functional change in the patch other than the log level.
---
 java/core/src/java/org/apache/orc/impl/RecordReaderImpl.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/java/core/src/java/org/apache/orc/impl/RecordReaderImpl.java b/java/core/src/java/org/apache/orc/impl/RecordReaderImpl.java
index 89343b9..19e8f92 100644
--- a/java/core/src/java/org/apache/orc/impl/RecordReaderImpl.java
+++ b/java/core/src/java/org/apache/orc/impl/RecordReaderImpl.java
@@ -210,7 +210,7 @@ public class RecordReaderImpl implements RecordReader {
     }
 
     this.noSelectedVector = !options.useSelected();
-    LOG.info("noSelectedVector={}", this.noSelectedVector);
+    LOG.debug("noSelectedVector={}", this.noSelectedVector);
     this.schema = evolution.getReaderSchema();
     this.path = fileReader.path;
     this.rowIndexStride = fileReader.rowIndexStride;
@@ -311,8 +311,8 @@ public class RecordReaderImpl implements RecordReader {
           expandCol = expandCol.getParent();
         }
       }
-      LOG.info("Filter Columns: " + filterColIds);
       this.startReadPhase = TypeReader.ReadPhase.LEADERS;
+      LOG.debug("Using startReadPhase: {} with filter columns: {}", startReadPhase, filterColIds);
     } else {
       this.startReadPhase = TypeReader.ReadPhase.ALL;
     }