Skip to content
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

[FLOSS] storage plugins: fail on unknown metadata #4555

Open
8 tasks done
markus2330 opened this issue Oct 15, 2022 · 3 comments
Open
8 tasks done

[FLOSS] storage plugins: fail on unknown metadata #4555

markus2330 opened this issue Oct 15, 2022 · 3 comments

Comments

@markus2330
Copy link
Contributor

markus2330 commented Oct 15, 2022

Currently most of the storage plugins simply eat unknown metadata. They should fail with an error instead.

I.e. kdb meta-set user:/mp blue blah

should either fail or kdb meta-get user:/mp blue should return blah

@Bujuhu
Copy link
Contributor

Bujuhu commented Oct 19, 2022

I've looked at every storage plugin except those, who discourage or omit mounting / setting values for keys altogether. I've created a single issue for every plugin I found which behaves that way. Creating a single issue for each one might seem kind of silly, seeing how similar they are, but I think it makes more sense that way, as most of the plugins probably won't be fixed all simultaneously.

@Bujuhu Bujuhu removed their assignment Oct 19, 2022
@Bujuhu Bujuhu removed the triage needed Issue needs clarifications. label Oct 19, 2022
@Bujuhu Bujuhu self-assigned this Oct 19, 2022
@markus2330
Copy link
Contributor Author

Great job! I linked all issues in the top post and ordered them by priority.

@flo91 flo91 changed the title storage plugins: fail on unknown metadata [FLOSS] storage plugins: fail on unknown metadata Oct 19, 2022
@markus2330
Copy link
Contributor Author

We should mark the buggy plugins or remove them. @atmaxinger what do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants