-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
use json-smart to parse the query #314
use json-smart to parse the query #314
Conversation
I guess, `JsonNode' shouldn't use the provider at all. |
This reverts parts of #101 |
Is this the only reason for JsonSmart to be a mandatory dependency? |
IMHO, Using json-smart as the default provider makes it a mandatory dependency, otherwise we ship a broken default config, don't we? |
In the |
I think it's common to exlude JsonSmart in some scenarios like Android. The main reason for this has been the ASM issues in JsonSmart. That should not be an issue now? |
Yes, that should be dealt with in #334. |
…-query-parsing use json-smart to parse the query
As I see it, the JsonProvider is only responsible to read the data, not the query.
See also #305, #275, but also #274.