Skip to content

Doc updates for metadata cleanup and storage - #12190

Merged
maytasm merged 7 commits into
apache:masterfrom
vtlim:docs-metadata
Jan 27, 2022
Merged

Doc updates for metadata cleanup and storage#12190
maytasm merged 7 commits into
apache:masterfrom
vtlim:docs-metadata

Conversation

@vtlim

@vtlim vtlim commented Jan 21, 2022

Copy link
Copy Markdown
Member

Doc updates to accompany config changes:

Doc changes, mostly to clean-metadata-store.md:

  • Describe settings that are enabled by default
  • Add section for how to disable cleanup and auto-kill
  • Better introduce topic of metadata storage, improve confusing points shared from the field
  • Improve organization and clarity in metadata-storage.md (no technical changes)

This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.

maytasm
maytasm requested changes Jan 25, 2022
View reviewed changes
Comment thread docs/dependencies/metadata-storage.md Outdated
Comment thread docs/operations/clean-metadata-store.md
Comment thread docs/operations/clean-metadata-store.md Outdated
vtlim
vtlim commented Jan 25, 2022
View reviewed changes
Comment thread docs/operations/clean-metadata-store.md
@vtlim
vtlim requested a review from maytasm January 25, 2022 20:56
@vtlim
vtlim requested a review from maytasm January 25, 2022 22:51
maytasm
maytasm reviewed Jan 26, 2022
View reviewed changes
Comment thread docs/operations/clean-metadata-store.md Outdated
maytasm
maytasm reviewed Jan 26, 2022
View reviewed changes
Comment thread docs/operations/clean-metadata-store.md Outdated
maytasm
maytasm reviewed Jan 26, 2022
View reviewed changes
Comment thread docs/operations/clean-metadata-store.md Outdated
maytasm
maytasm reviewed Jan 26, 2022
View reviewed changes
Comment thread docs/operations/clean-metadata-store.md Outdated
maytasm
maytasm requested changes Jan 26, 2022
View reviewed changes

@maytasm maytasm 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.

Please see comments.

@vtlim @maytasm
Apply suggestions from code review
b3d190f 
Co-authored-by: Maytas Monsereenusorn <maytasm@apache.org>
@maytasm
maytasm merged commit 24716bf into apache:master Jan 27, 2022
@vtlim
vtlim deleted the docs-metadata branch January 27, 2022 20:47
@abhishekagarwal87 abhishekagarwal87 added this to the 0.23.0 milestone May 11, 2022
riovic918data pushed a commit to riovic918data/druid that referenced this pull request Jun 12, 2026
@vtlim @maytasm
Doc updates for metadata cleanup and storage (apache#12190)
a6dbff2 
* doc updates for metadata storage/cleanup

* Add comments for disabling cleanup

* Apply suggestions from code review

* updated for apache#12201

* Apply suggestions from code review

Co-authored-by: Maytas Monsereenusorn <maytasm@apache.org>

* move retention period line earlier; more concise text

* fix typo

Co-authored-by: Maytas Monsereenusorn <maytasm@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@maytasm maytasm maytasm approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

0.23.0

Development

Successfully merging this pull request may close these issues.

3 participants

@vtlim @maytasm @abhishekagarwal87