Skip to content

getCompactionTaskCapacity is not worker category aware. #15847

Open
@m-ghazanfar

Description

@m-ghazanfar

Description

The getCompactionTaskCapacity function which is used to ascertain that the Druid cluster has enough task slots before the coordinator schedules additional compaction tasks doesn't take into consideration overlord dynamic config. The overlord dynamic config can prevent compaction tasks from running on specific categories of workers. This way, the compaction task capacity is incorrectly overestimated.

For example, I have two worker categories, compaction-category with a total of 600 task slots and another ingestion-category with 2000 slots(high number because of multiple ingestion task replicas).

Using the overlord dynamic config,

  • compaction-category is configured to run the following task types,

    • kill
    • compact
    • single_phase_sub_task
    • partial_dimension_cardinality
    • partial_index_generate
    • partial_index_generic_merge
  • ingestion-category is configured to run,

    • index_kafka

Now, getCompactionTaskCapacity would return 2600 as the total capacity, which is inaccurate since only 600 slots are actually available for compaction tasks. While this might not pose a problem in a healthy cluster, it becomes critical during compaction task failures. The oversight leads to the coordinator creating excessive compaction tasks, resulting in contention on compaction slots and slowing down all compaction tasks. This creates a feedback loop where the increasing number of compaction tasks exacerbates contention, ultimately overwhelming the overlord with too many tasks to handle

Affected Version

Saw this on Druid 25. Is also present in master.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions