kfaraz commented on PR #18968:
URL: https://github.com/apache/druid/pull/18968#issuecomment-3846703954
> Currently, CompactionStatus conflates two concerns:
Status: What's the task status of a CompactionCandidate?
Eligibility: Should we compact this interval now, and how?
@cecemei , just so that we are on the same page, the `CompactionStatus` does
not exactly contain information of the compaction "task" (except for the
RUNNING state, which we can now remove in this PR). It contains info of the
current state of compaction in the interval, i.e. number of compacted segments,
number of uncompacted segments and overall state (COMPLETE/PENDING/SKIPPED).
It would be nice to get rid of the SKIPPED state too since it is more of an
eligibility thing, but I guess we can leave it for now.
--
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]
