Skip to content

docs: revise concurrent append and replace#15760

Merged
317brian merged 8 commits intoapache:masterfrom
317brian:copyedit-conc-apprepl
Feb 1, 2024
Merged

docs: revise concurrent append and replace#15760
317brian merged 8 commits intoapache:masterfrom
317brian:copyedit-conc-apprepl

Conversation

@317brian
Copy link
Contributor

@317brian 317brian commented Jan 25, 2024

  • Moves most of the concurrent append and replace stuff out of automatic compaction into its own page in the ingestion section
  • Revises the steps to use the new context parameter instead of manually setting the lock type
  • Hides the older but potentially still useful info about the lock types in a collapsed section

Rendered preview available here: https://druid-git-copyedit-conc-apprepl-317brian.vercel.app/docs/latest/ingestion/concurrent-append-replace

This PR has:

  • been self-reviewed.

@317brian 317brian marked this pull request as ready for review January 25, 2024 21:06
317brian and others added 3 commits January 25, 2024 14:36
Co-authored-by: Victoria Lim <vtlim@users.noreply.github.com>
@317brian 317brian requested a review from vtlim January 26, 2024 23:09
Copy link
Member

@vtlim vtlim left a comment

Choose a reason for hiding this comment

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

Some nits. Otherwise looks good! 🦖

Co-authored-by: Victoria Lim <vtlim@users.noreply.github.com>
@317brian 317brian merged commit 6d617c3 into apache:master Feb 1, 2024
317brian added a commit to 317brian/druid that referenced this pull request Feb 1, 2024
Co-authored-by: Victoria Lim <vtlim@users.noreply.github.com>
(cherry picked from commit 6d617c3)
@adarshsanjeev adarshsanjeev added this to the 30.0.0 milestone May 6, 2024
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