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

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


The following commit(s) were added to refs/heads/ty_new_vector by this push:
     new 674a88c  fix sonar bug
674a88c is described below

commit 674a88c73713ee4c07ecf8eb06a779cafeb6cab1
Author: JackieTien97 <[email protected]>
AuthorDate: Sat Nov 20 09:12:29 2021 +0800

    fix sonar bug
---
 .../org/apache/iotdb/db/query/reader/chunk/MemAlignedPageReader.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/server/src/main/java/org/apache/iotdb/db/query/reader/chunk/MemAlignedPageReader.java
 
b/server/src/main/java/org/apache/iotdb/db/query/reader/chunk/MemAlignedPageReader.java
index 7dcd5ec..291b4d4 100644
--- 
a/server/src/main/java/org/apache/iotdb/db/query/reader/chunk/MemAlignedPageReader.java
+++ 
b/server/src/main/java/org/apache/iotdb/db/query/reader/chunk/MemAlignedPageReader.java
@@ -69,8 +69,8 @@ public class MemAlignedPageReader implements IPageReader, 
IAlignedPageReader {
       // or current row is not satisfied with the filter, just discard it
       // TODO fix value filter firstNotNullObject, currently, if it's a value 
filter, it will only
       // accept AlignedPath with only one sub sensor
-      if (firstNotNullObject != null && valueFilter == null
-          || valueFilter.satisfy(timeValuePair.getTimestamp(), 
firstNotNullObject)) {
+      if (firstNotNullObject != null && (valueFilter == null || 
valueFilter.satisfy(
+          timeValuePair.getTimestamp(), firstNotNullObject))) {
         batchData.putVector(timeValuePair.getTimestamp(), values);
       }
     }

Reply via email to