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

Uniform jsonQuery and jsonPath #746

Open
ccl-core opened this issue Sep 25, 2024 · 0 comments
Open

Uniform jsonQuery and jsonPath #746

ccl-core opened this issue Sep 25, 2024 · 0 comments

Comments

@ccl-core
Copy link
Contributor

In the croissant specs, we use two different terminology: jsonQuery for transforms (https://docs.mlcommons.org/croissant/docs/croissant-spec.html#transform) and jsonPath for extract (https://docs.mlcommons.org/croissant/docs/croissant-spec.html#extract).

The latter is defined here: https://goessner.net/articles/JsonPath/ . I don't know if the former defines a different query language.

It is not really clear what the difference between the two is, given that both can be used interchangeably (see e.g. a Transform operation using JsonPath here).

Possibly, renaming JsonQuery to JsonPath in the specs could avoid redundancy and make the definitions clearer.

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

No branches or pull requests

1 participant