asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Till Westmann (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: reduce log output during tests
Date Sat, 02 Apr 2016 07:38:42 GMT
Till Westmann has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/773

Change subject: reduce log output during tests
......................................................................

reduce log output during tests

Change-Id: Ia6a114e59505341e46f9cb50d3850190630b64d6
---
M asterix-external-data/src/main/java/org/apache/asterix/external/operators/FeedMetaComputeNodePushable.java
M asterix-external-data/src/test/java/org/apache/asterix/external/classad/test/ClassAdUnitTest.java
2 files changed, 10 insertions(+), 4 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/73/773/1

diff --git a/asterix-external-data/src/main/java/org/apache/asterix/external/operators/FeedMetaComputeNodePushable.java
b/asterix-external-data/src/main/java/org/apache/asterix/external/operators/FeedMetaComputeNodePushable.java
index 210a94d..6c9ef8d 100644
--- a/asterix-external-data/src/main/java/org/apache/asterix/external/operators/FeedMetaComputeNodePushable.java
+++ b/asterix-external-data/src/main/java/org/apache/asterix/external/operators/FeedMetaComputeNodePushable.java
@@ -206,15 +206,21 @@
                 inputSideHandler.setFinished(true);
             }
             coreOperator.close();
-            System.out.println("CLOSED " + coreOperator + " STALLED ?" + stalled + " ENDED
" + end);
+            if (LOGGER.isLoggable(Level.FINE)) {
+                LOGGER.fine("CLOSED " + coreOperator + " STALLED ?" + stalled + " ENDED "
+ end);
+            }
         } catch (InterruptedException e) {
             throw new HyracksDataException(e);
         } finally {
             if (!stalled) {
                 deregister();
-                System.out.println("DEREGISTERING " + this.feedRuntime.getRuntimeId());
+                if (LOGGER.isLoggable(Level.FINE)) {
+                    LOGGER.fine("DEREGISTERING " + this.feedRuntime.getRuntimeId());
+                }
             } else {
-                System.out.println("NOT DEREGISTERING " + this.feedRuntime.getRuntimeId());
+                if (LOGGER.isLoggable(Level.FINE)) {
+                    LOGGER.fine("NOT DEREGISTERING " + this.feedRuntime.getRuntimeId());
+                }
             }
             inputSideHandler.close();
             if (LOGGER.isLoggable(Level.INFO)) {
diff --git a/asterix-external-data/src/test/java/org/apache/asterix/external/classad/test/ClassAdUnitTest.java
b/asterix-external-data/src/test/java/org/apache/asterix/external/classad/test/ClassAdUnitTest.java
index 32ce485..572460e 100644
--- a/asterix-external-data/src/test/java/org/apache/asterix/external/classad/test/ClassAdUnitTest.java
+++ b/asterix-external-data/src/test/java/org/apache/asterix/external/classad/test/ClassAdUnitTest.java
@@ -49,7 +49,7 @@
      * Rigourous Test :-)
      */
     public void test() {
-        String[] args = { "", "-d", "-vv", "-all" };
+        String[] args = { "", "-d", "-all" };
         try {
             ClassAdUnitTester.test(args.length, args, new ClassAdObjectPool());
         } catch (Throwable e) {

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/773
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia6a114e59505341e46f9cb50d3850190630b64d6
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <tillw@apache.org>

Mime
View raw message