You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iotdb.apache.org by qi...@apache.org on 2020/11/11 03:31:47 UTC

[iotdb] branch master updated: Fix code smell (#2004)

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

qiaojialin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/iotdb.git


The following commit(s) were added to refs/heads/master by this push:
     new feb2001  Fix code smell (#2004)
feb2001 is described below

commit feb20010acb728369e098e79ae23e367a82823a1
Author: zhanglingzhe0820 <44...@qq.com>
AuthorDate: Wed Nov 11 11:31:29 2020 +0800

    Fix code smell (#2004)
---
 .../db/engine/storagegroup/StorageGroupProcessor.java | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/server/src/main/java/org/apache/iotdb/db/engine/storagegroup/StorageGroupProcessor.java b/server/src/main/java/org/apache/iotdb/db/engine/storagegroup/StorageGroupProcessor.java
index 2b015b2..f581489 100755
--- a/server/src/main/java/org/apache/iotdb/db/engine/storagegroup/StorageGroupProcessor.java
+++ b/server/src/main/java/org/apache/iotdb/db/engine/storagegroup/StorageGroupProcessor.java
@@ -1389,10 +1389,9 @@ public class StorageGroupProcessor {
       throws MetadataException {
 
     if (config.isDebugOn()) {
-      DEBUG_LOGGER.info(
-          "Path: " + deviceId.getFullPath() + TsFileConstant.PATH_SEPARATOR + measurementId
-              + ", get tsfile list: " + tsFileResources + " isSeq: " + isSeq + " timefilter: " + (
-              timeFilter == null ? "null" : timeFilter.toString()));
+      DEBUG_LOGGER
+          .info("Path: {}.{}, get tsfile list: {} isSeq: {} timefilter: {}", deviceId.getFullPath(),
+              measurementId, tsFileResources, isSeq, (timeFilter == null ? "null" : timeFilter));
     }
 
     MeasurementSchema schema = IoTDB.metaManager.getSeriesSchema(deviceId, measurementId);
@@ -1444,8 +1443,8 @@ public class StorageGroupProcessor {
       Filter timeFilter, boolean isSeq) {
     if (!tsFileResource.containsDevice(deviceId)) {
       if (config.isDebugOn()) {
-        DEBUG_LOGGER.info("Path: " + deviceId + " file " + tsFileResource
-            + " is not satisfied because of no device!");
+        DEBUG_LOGGER.info("Path: {} file {} is not satisfied because of no device!", deviceId,
+            tsFileResource);
       }
       return false;
     }
@@ -1457,8 +1456,8 @@ public class StorageGroupProcessor {
 
     if (!isAlive(endTime)) {
       if (config.isDebugOn()) {
-        DEBUG_LOGGER.info("Path: " + deviceId + " file " + tsFileResource
-            + " is not satisfied because of ttl!");
+        DEBUG_LOGGER
+            .info("Path: {} file {} is not satisfied because of ttl!", deviceId, tsFileResource);
       }
       return false;
     }
@@ -1466,8 +1465,8 @@ public class StorageGroupProcessor {
     if (timeFilter != null) {
       boolean res = timeFilter.satisfyStartEndTime(startTime, endTime);
       if (config.isDebugOn() && !res) {
-        DEBUG_LOGGER.info("Path: " + deviceId + " file " + tsFileResource
-            + " is not satisfied because of time filter!");
+        DEBUG_LOGGER.info("Path: {} file {} is not satisfied because of time filter!", deviceId,
+            tsFileResource);
       }
       return res;
     }