-
Notifications
You must be signed in to change notification settings - Fork 25
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
Union with wildcard and number in Proposal A #51
Comments
BTW, the background to this is that I'm bumping |
I wonder if part of the rationale is that Proposal A is free to extend the consensus in the sense that if the consensus on a given selector is |
I vote with both hands for union of arbitrary matchers (numbers, identifiers, wildcards and filters) and for "extending consensus" principle. The union approach is powerful and intuitive. |
Speaking about extending consensus - we should choose one of the following as the final goal of Proposal A:
As for me I think that in current situation (lots of different implementation with no standard) position 2 is more reasonable. |
Meta-comment: I think we could benefit from less structured discussion in addition to issues, so I've taken the liberty of creating a jsonpath-standard slack workspace. This invitation should work for 30 days. |
I see a few things discussed here:
I'll try to answer 1. Let's continue to talk about 2. How about we discuss 3. in a new issue? I didn't think about 3 at all, so thanks for bringing it up! |
Proposal A supports a mixture of wildcards and numbers such as
$[*,1]
whereas the consensus is "not supported".What's the rationale for this behaviour of Proposal A? Should Proposal A change?
My personal preference is to go further and allow arbitrary mixtures of wildcards and numbers and produce the corresponding values in the output as that seems to make the syntax and semantics more uniform. For example, Goessner (here) allows the selector
$[*,1,0,*]
which, when applied to the input document:produces the output:
Admittedly, this behaviour is useless in practice, but it should makes the behaviour easier to understand (and document) as there are fewer special cases to remember.
The text was updated successfully, but these errors were encountered: