fanhualta commented on a change in pull request #3191:
URL: https://github.com/apache/iotdb/pull/3191#discussion_r639873818
##########
File path:
server/src/main/java/org/apache/iotdb/db/engine/storagegroup/TsFileResource.java
##########
@@ -810,11 +810,11 @@ public void updatePlanIndexes(TsFileResource another) {
}
public boolean isPlanIndexOverlap(TsFileResource another) {
- return another.maxPlanIndex >= this.minPlanIndex && another.minPlanIndex
<= this.maxPlanIndex;
+ return another.maxPlanIndex > this.minPlanIndex && another.minPlanIndex <
this.maxPlanIndex;
}
public boolean isPlanRangeCovers(TsFileResource another) {
- return this.minPlanIndex <= another.minPlanIndex && another.maxPlanIndex
<= this.maxPlanIndex;
+ return this.minPlanIndex < another.minPlanIndex && another.maxPlanIndex <
this.maxPlanIndex;
}
Review comment:
I think it's necessary. When data is migrated, each file to be loaded
has to reset the plan index. which will be set as max log index in log manager,
so there will be multiple files with the same plan index.
--
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]
