Skip to content
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

Slow processing? #16

Open
jamesladd opened this issue Mar 20, 2018 · 2 comments
Open

Slow processing? #16

jamesladd opened this issue Mar 20, 2018 · 2 comments

Comments

@jamesladd
Copy link

Hi dvdLn,

I think this is a really useful package. Thanks for taking the time to write it.

When using jsonpath-object-transform V1.0.4 I have noticed that the time taken to perform
a transform is quite long. Of course this is dependent on how long my JSON is and the number of rules
to apply to it.

However, is there anything you can do to improve the speed of this library?

Are there any tips about how to improve the speed of transforms on my side, ie: particular expressions
to avoid ?

  • James.
@deekayw0n
Copy link

Bump.

It looks like jsonpath-object-transform v1.0.4 depends on JSONPath which has known performance issues (at least at this time): JSONPath-Plus/JSONPath#14.

@dvdln - you may want to consider using latest jsonpath as a dependency instead: https://www.npmjs.com/package/jsonpath. Were you using any additional features that JSONPath was providing?

Trying to find the perfect blend of JSON transformation packages--thought in 2018 there would be some clear winners. This one does seem very promising though!

Cheers
Damian

@brettz9
Copy link

brettz9 commented Jun 16, 2019

FWIW, as a (relatively passive) maintainer of jsonpath-plus, it has come to our attention that reports of jsonpath-plus' supposed performance problems may have been premature (or tied to earlier Node versions) with it seeming to actually perform well compared to other jsonpath implementations. See JSONPath-Plus/JSONPath#14 for a fuller background. If you find this to be inaccurate, however, it would be great to have additional tests that indicate any problems submitted to the likes of https://github.com/andykais/json-querying-performance-testing , so the libraries can improve and users may discover the best solution for their needs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants