EmbeddedRocksDB: Go back to using data/ path by default by Algunenano · Pull Request #87463 · ClickHouse/ClickHouse · GitHub
Skip to content

EmbeddedRocksDB: Go back to using data/ path by default#87463

Merged
Algunenano merged 1 commit into
ClickHouse:masterfrom
Algunenano:rock_default
Sep 23, 2025
Merged

EmbeddedRocksDB: Go back to using data/ path by default#87463
Algunenano merged 1 commit into
ClickHouse:masterfrom
Algunenano:rock_default

Conversation

@Algunenano

Copy link
Copy Markdown
Member

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):

EmbeddedRocksDB: Go back to using data/ path by default

Follow up to #87109 and #87392.
Internal discussion confirmed that the preferred default path (when it's not provided) should be under data/ as it was, and only use and enforce user_files for explicitly declared paths.

Not for changelog since it's unreleased and will be backported with the rest of the security fixes.

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@Algunenano Algunenano self-assigned this Sep 23, 2025
@Algunenano Algunenano added the pr-must-backport Pull request should be backported intentionally. Use this label with great care! label Sep 23, 2025
@clickhouse-gh

clickhouse-gh Bot commented Sep 23, 2025

Copy link
Copy Markdown
Contributor

@clickhouse-gh clickhouse-gh Bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Sep 23, 2025
@Algunenano

Copy link
Copy Markdown
Member Author

@Algunenano Algunenano added this pull request to the merge queue Sep 23, 2025
Merged via the queue into ClickHouse:master with commit 8f36acb Sep 23, 2025
114 of 122 checks passed
@Algunenano Algunenano deleted the rock_default branch September 23, 2025 13:07
robot-clickhouse added a commit that referenced this pull request Sep 23, 2025
robot-clickhouse added a commit that referenced this pull request Sep 23, 2025
robot-clickhouse added a commit that referenced this pull request Sep 23, 2025
robot-clickhouse added a commit that referenced this pull request Sep 23, 2025
robot-clickhouse added a commit that referenced this pull request Sep 23, 2025
@robot-ch-test-poll robot-ch-test-poll added the pr-synced-to-cloud The PR is synced to the cloud repo label Sep 23, 2025
@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Sep 23, 2025
clickhouse-gh Bot added a commit that referenced this pull request Sep 23, 2025
Backport #87463 to 25.6: EmbeddedRocksDB: Go back to using data/ path by default
clickhouse-gh Bot added a commit that referenced this pull request Sep 23, 2025
Backport #87463 to 25.8: EmbeddedRocksDB: Go back to using data/ path by default
@robot-ch-test-poll4 robot-ch-test-poll4 added pr-backports-created-cloud deprecated label, NOOP pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR labels Sep 23, 2025
Algunenano added a commit that referenced this pull request Sep 23, 2025
Backport #87463 to 25.3: EmbeddedRocksDB: Go back to using data/ path by default
Algunenano added a commit that referenced this pull request Sep 23, 2025
Backport #87463 to 25.9: EmbeddedRocksDB: Go back to using data/ path by default
Algunenano added a commit that referenced this pull request Sep 23, 2025
Backport #87463 to 25.7: EmbeddedRocksDB: Go back to using data/ path by default
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-backports-created-cloud deprecated label, NOOP pr-must-backport Pull request should be backported intentionally. Use this label with great care! pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR pr-not-for-changelog This PR should not be mentioned in the changelog pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants