| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
@arouel thanks very much, I have fix the code based on your suggestion. Benchmark testing indicators. refer: Result "org.simdjson.AParseAndSelectFixPathBenchMark.parseMultiValuesForFixPaths_Jackson": Result "org.simdjson.ParseAndSelectFixPathBenchMark.parseMultiValuesForFixPaths_SimdJson": Result "org.simdjson.ParseAndSelectFixPathBenchMark.parseMultiValuesForFixPaths_SimdJsonParserWithFixPath": |
Sorry, something went wrong.
|
How is this different from On-Demand parsing available in the c++ simdjson version? I introduced a form of on-demand parsing in #51 (see: org.simdjson.OnDemandJsonIterator). The API requires specifying a target class to which the JSON will be parsed. However, it should be relatively easy to extend this to support a DOM-like API (JsonValue, JsonIterator, etc.), which I believe is more intuitive than introducing syntax for accessing fields and then returning an array of strings with the corresponding values. |
Sorry, something went wrong.
|
@piotrrzysko I agree with you, a DOM-like API (JsonValue, JsonIterator, etc.) would be very helpful in use cases where only specific parts of the JSON are conditionally relevant, so that a mapping to an object would cause allocation that you want to avoid. Can you guide us a bit, so that we can prepare a PR? |
Sorry, something went wrong.
There was a problem hiding this comment.
@heykirby I just want share some thoughts/questions:
With some minor API changes in simdjson-java, could we keep the SimdJsonParserWithFixPath in another codebase or it could life in a contribution module, because it is tailored for a very specific use case?
Isn't a record JsonNode sufficient compared to using lombok?
Sorry, something went wrong.
@arouel Thanks arouel,the unused imports has been removed |
Sorry, something went wrong.
hello, piotrrzysko, I used on-demand parsing,it is very convenient and efficient to deserialize json strings into java classes.it is also a solution provided by many mainstream json sdk. For SimdJsonParserWithFixPath, if we want get values for multi-paths: [$.a.c,$.a,$.a.d,$.b], we only need to provide the json paths, the usage is similar to hive's user define function: json_tuple. It also supports obtaining the value of the children of the container object while obtaining the compressed string value of the container object. |
Sorry, something went wrong.
|
Hi, sorry for the delayed reply. @heykirby The schema-based API you’re referring to is simply using logic that could potentially be utilized by the on-demand DOM API as well.
I’d be happy to help. Perhaps I could start by creating a skeleton of the on-demand DOM API. |
Sorry, something went wrong.
thanks,piotrrzysko, It's always an expected feature. |
Sorry, something went wrong.
|
@piotrrzysko I submitted a new PR, could you give me some guidance? #63 |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
issue: #59