Yingchun Lai has posted comments on this change. ( http://gerrit.cloudera.org:8080/18503 )
Change subject: [KUDU-3367][compaction] Fix delta file with full of delete op can not be schedule to compact ...................................................................... Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/18503/2//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/18503/2//COMMIT_MSG@15 PS2, Line 15: So I added a flag FLAGS_ignore_delete_op_for_compact to solve this situation. : When we found a similar situation, we can schedule the compact algorithm to deal : with REDO deltas by set FLAGS_ignore_delete_op_for_compact to false. When we solve : this problem, we need to restore the flag to true to ensure that the compact : performance will not be affected. > Do you mean we don't need this flag to keep this function on, just keep thi Maybe add a flag such as FLAGS_check_delete_op_for_compact_ratio, it's ranged in [0,1], indicate how offen to check, the default value can be very low to reduce the affect on performance. -- To view, visit http://gerrit.cloudera.org:8080/18503 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8b26737dffecc17688b42188da959b2ba16351ed Gerrit-Change-Number: 18503 Gerrit-PatchSet: 3 Gerrit-Owner: KeDeng <[email protected]> Gerrit-Reviewer: KeDeng <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Comment-Date: Tue, 10 May 2022 06:47:51 +0000 Gerrit-HasComments: Yes
