Skip to content

Change StorageType to an open enumeration #1997

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 16, 2022
Merged

Change StorageType to an open enumeration #1997

merged 1 commit into from
Dec 16, 2022

Conversation

swallez
Copy link
Member

@swallez swallez commented Dec 12, 2022

The StorageType enumeration is non exhaustive: some plugins may add additional storage types, and snapshot/frozen repositories, although they cannot be created directly, are identified with a storage type snapshot.

Reported in elastic/elasticsearch-java#458

Copy link
Contributor

@Anaethelion Anaethelion left a comment

Choose a reason for hiding this comment

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

LGTM!

@swallez swallez merged commit 1dcc9a7 into main Dec 16, 2022
@swallez swallez deleted the open-storage-type branch December 16, 2022 13:13
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.

2 participants