Skip to content

Added concurrent compaction switches#15114

Merged
AmatyaAvadhanula merged 5 commits into
apache:masterfrom
lorem--ipsum:concurrent-compaction-ui
Oct 13, 2023
Merged

Added concurrent compaction switches#15114
AmatyaAvadhanula merged 5 commits into
apache:masterfrom
lorem--ipsum:concurrent-compaction-ui

Conversation

@lorem--ipsum

Copy link
Copy Markdown
Contributor

This PR adds checkboxes in two places:

  • in the classic batch ingestion wizard
image
  • in the compaction configuration UI
image

These both set the context.taskLockType accordingly, to either 'APPEND', 'REPLACE' or undefined based on the checkbox state and the current spec.

@LakshSingla LakshSingla added this to the 28.0 milestone Oct 10, 2023
Comment thread web-console/src/views/load-data-view/load-data-view.tsx Outdated
@AmatyaAvadhanula

Copy link
Copy Markdown
Contributor

@lorem--ipsum, could you please also add a switch in the streaming ingestion wizard that adds an APPEND lock to the context when concurrent compaction is enabled?

@AmatyaAvadhanula AmatyaAvadhanula left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @lorem--ipsum, LGTM!

@AmatyaAvadhanula
AmatyaAvadhanula merged commit 9ca10c7 into apache:master Oct 13, 2023
LakshSingla pushed a commit to LakshSingla/druid that referenced this pull request Oct 14, 2023
* Added concurrent compaction switches
AmatyaAvadhanula pushed a commit that referenced this pull request Oct 14, 2023
* Added concurrent compaction switches

Co-authored-by: Sébastien <[email protected]>
ektravel pushed a commit to ektravel/druid that referenced this pull request Oct 16, 2023
* Added concurrent compaction switches
CaseyPan pushed a commit to CaseyPan/druid that referenced this pull request Nov 17, 2023
* Added concurrent compaction switches
riovic918data pushed a commit to riovic918data/druid that referenced this pull request Jun 12, 2026
* Added concurrent compaction switches
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants