You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@iotdb.apache.org by GitBox <gi...@apache.org> on 2022/12/29 03:05:18 UTC

[GitHub] [iotdb] xingtanzjr commented on a diff in pull request #8615: [IOTDB-5286]Fix target files should not be select to participate in other compaction task

xingtanzjr commented on code in PR #8615:
URL: https://github.com/apache/iotdb/pull/8615#discussion_r1058704789


##########
server/src/main/java/org/apache/iotdb/db/engine/storagegroup/TsFileResource.java:
##########
@@ -179,6 +179,14 @@ public TsFileResource(File file) {
     this.timeIndex = CONFIG.getTimeIndexLevel().getTimeIndex();
   }
 
+  /** Used for compaction to create target files. */
+  public TsFileResource(File file, TsFileResourceStatus status) {
+    this.file = file;

Review Comment:
   using `this(file)` to reduce duplicated codes



-- 
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.

To unsubscribe, e-mail: reviews-unsubscribe@iotdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org