Skip to content

Conversation

@andrewbaptist
Copy link

No description provided.

@blathers-crl
Copy link

blathers-crl bot commented Jan 31, 2025

Your pull request contains more than 1000 changes. It is strongly encouraged to split big PRs into smaller chunks.

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@andrewbaptist andrewbaptist force-pushed the 2025-01-31-convert-store-spec branch 3 times, most recently from fb19eb5 to 957a45d Compare January 31, 2025 23:43
Andrew Baptist added 2 commits February 3, 2025 09:06
Previously the ExternalIODir was stored as a variable inside the
BaseConfig struct and updated based on the StoreSpecs if it wasn't
explicitly set. This commit moves it into the StorageConfig and removes
it from the BaseConfig.

Epic: CRDB-41111

Release note: None
This commit converts the StoreSpecs to a protobuf structure and moves it
into the storagepb directory. This is now part of the StorageConfig
structure.

Epic: none

Release note: None
@andrewbaptist andrewbaptist force-pushed the 2025-01-31-convert-store-spec branch from 957a45d to 25229c2 Compare February 3, 2025 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants