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

V3 TBD: re-discuss "must understand". #88

Closed
Carreau opened this issue Aug 24, 2020 · 2 comments
Closed

V3 TBD: re-discuss "must understand". #88

Carreau opened this issue Aug 24, 2020 · 2 comments
Labels
core-protocol-v3.0 Issue relates to the core protocol version 3.0 spec

Comments

@Carreau
Copy link
Contributor

Carreau commented Aug 24, 2020

The "extensions" section of v3 states:

An array containing zero or more objects, each of which identifies
a protocol extension and provides any additional extension
configuration metadata. Each object must contain the name
``extension`` whose value is a URI that identifies a Zarr protocol
extension and dereferences to a human readable representation of
the extension specification. Each object must also contain the
name ``must_understand`` whose value is either the literal
``true`` or ``false``. Each object may also contain the name
``configuration`` whose value is defined by the protocol
extension.

If an implementation of this specification encounters an extension
that it does not recognize, but the value of ``must_understand``
is ``false``, then the extension may be ignored and processing may
continue. If the extension is not recognized and the value of
``must_understand`` is ``true`` then processing must terminate and
an appropriate error raised.

From previous discussions it seem like we agree "must understand" seem to restrictive, for example an extension may affect only a subset of arrays, or be required only to write array.

@Carreau Carreau added the core-protocol-v3.0 Issue relates to the core protocol version 3.0 spec label Sep 25, 2020
@Carreau
Copy link
Contributor Author

Carreau commented Sep 25, 2020

Add a note that this is one of the area where feedback is wanted during RFC.

@jstriebel
Copy link
Member

From previous discussions it seem like we agree "must understand" seem to restrictive, for example an extension may affect only a subset of arrays, or be required only to write array.

The current mechanism allows to specify array-specific extensions per array, and not on the hierarchy level. My understanding is, that a zarr client could still open the hierarchy, even if some of the must-understand array extensions are not understood, and just could not open the affected arrays. Closing this therefore for now, please re-open if you feel this is not resolved @Carreau.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core-protocol-v3.0 Issue relates to the core protocol version 3.0 spec
Projects
Status: Done
Development

No branches or pull requests

2 participants