GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||
GeezerVison is showing github.com — add SimdJsonParser2 base on bitindex by heykirby · Pull Request #60 · simdjson/simdjson-java Conversationissue: #59
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
heykirby
force-pushed
the
feature_simdjson2
branch
5 times, most recently
from
5c92d47 to
3139b2c
Compare
arouel
reviewed
Uh 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
force-pushed
the
feature_simdjson2
branch
from
e84e78d to
e2f0def
Compare
|
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": |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
e2f0def to
4bed300
Compare
|
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: |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@piotrrzysko I agree with you, a DOM-like API ( Can you guide us a bit, so that we can prepare a PR? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was a problem 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 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.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
ecd8e0e to
204fed7
Compare
There was an error while loading. Please reload this page.
@arouel Thanks arouel,the unused imports has been removed |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
204fed7 to
f6fc9e5
Compare
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.
There was an error while loading. Please reload this page.
|
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.
There was an error while loading. Please reload this page.
thanks,piotrrzysko, It's always an expected feature. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@piotrrzysko I submitted a new PR, could you give me some guidance? #63 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.