github-actions[bot] commented on code in PR #65373:
URL: https://github.com/apache/doris/pull/65373#discussion_r3567915910
##########
be/src/cloud/cloud_storage_engine.cpp:
##########
@@ -1072,6 +1075,8 @@ Status
CloudStorageEngine::_submit_cumulative_compaction_task(const CloudTabletS
long now =
duration_cast<milliseconds>(system_clock::now().time_since_epoch())
.count();
// sleep 5s for this tablet
+ tablet->set_last_cumu_compaction_status(
Review Comment:
This delay branch now records a fresh status, but it still runs after
`_request_tablet_global_compaction_lock()` has prepared a cumulative compaction
job in the meta-service. Returning here only removes the compaction from the BE
maps; it never calls
`CloudCumulativeCompaction::garbage_collection()`/`abort_tablet_job()`, and
after `erase_executing_cumu_compaction()` the lease thread no longer renews the
job. Until the meta-service removes that unexpired job by lease/expiration, the
next overlapping compaction sees `JOB_TABLET_BUSY` even though `/run_status` no
longer reports this task locally. Please abort the prepared job on this delay
path, or decide to delay before requesting the global compaction lock.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
