- Notifications
You must be signed in to change notification settings - Fork34
add SimdJsonParser2 base on bitindex#60
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
5c92d47 to3139b2cCompareUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
heykirby commentedOct 19, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this 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": |
piotrrzysko commentedOct 20, 2024
How is this different fromOn-Demand parsing available in the c++ simdjson version? I introduced a form of on-demand parsing in#51 (see: |
arouel commentedOct 21, 2024
@piotrrzysko I agree with you, a DOM-like API ( Can you guide us a bit, so that we can prepare a PR? |
arouel left a comment• edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
@heykirby I just want share some thoughts/questions:
With some minor API changes insimdjson-java, could we keep theSimdJsonParserWithFixPath in another codebase or it could life in a contribution module, because it is tailored for a very specific use case?
Isn't arecord JsonNode sufficient compared to usinglombok?
Uh oh!
There was an error while loading.Please reload this page.
heykirby commentedOct 22, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@arouel Thanks arouel,the unused imports has been removed |
heykirby commentedNov 11, 2024
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. |
piotrrzysko commentedNov 22, 2024
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. |
heykirby commentedNov 24, 2024
thanks,piotrrzysko, It's always an expected feature. |
heykirby commentedNov 26, 2024
@piotrrzysko I submitted a new PR, could you give me some guidance?#63 |

issue:#59