JackieTien97 commented on a change in pull request #460:
URL: https://github.com/apache/iotdb/pull/460#discussion_r536745981
##########
File path:
tsfile/src/main/java/org/apache/iotdb/tsfile/read/reader/page/PageReader.java
##########
@@ -162,7 +163,11 @@ public Statistics getStatistics() {
@Override
public void setFilter(Filter filter) {
- this.filter = filter;
+ if (this.filter == null) {
Review comment:
Actually, i think we should just use the filter to overwrite the former
one. The former one is actually a time filter and it has been checked when the
PageReader is constructed.
And alse don't forget to change back the MemPageReader
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
