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

Abstract store interface should be clarified as descriptive, not normative #307

Open
jhamman opened this issue Aug 21, 2024 · 3 comments · May be fixed by #308
Open

Abstract store interface should be clarified as descriptive, not normative #307

jhamman opened this issue Aug 21, 2024 · 3 comments · May be fixed by #308

Comments

@jhamman
Copy link
Member

jhamman commented Aug 21, 2024

I would like to understand why the spec includes a Abstract store interface section and ask if this section should be clarified as descriptive, not normative.

Why should the spec care what interface and implementation chooses for the Store abstraction? So long as the store does the right thing, surely the implementation is free to choose a different abstract interface?

@d-v-b
Copy link
Contributor

d-v-b commented Aug 22, 2024

the normativity of that section also stood out to me. I don't know why we need normativity here; in fact, it's probably healthy to allow different implementations to use whatever internal APIs seem appropriate. If there is a "right way" to implement zarr, then it can probably only be discovered empirically (and I doubt there is a "right way").

@dcherian
Copy link

FWIW the wording is strong:

For a store to be compatible with this specification, it must support a set of operations defined in the Abstract store interface subsection.

@jbms
Copy link
Contributor

jbms commented Aug 22, 2024

Agreed that this is not intended to specify the actual API in any implementation.

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 a pull request may close this issue.

4 participants