-
Notifications
You must be signed in to change notification settings - Fork 65
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
Add Jsonpath
support for Ballerina json module
#5938
Add Jsonpath
support for Ballerina json module
#5938
Comments
Currently working on the third subtask. I compared the output module of the first Jayway Implementation and the SPI Implementation task. Currently working a way to optimize |
The reason for this performance issue is, |
But if we consider some basic queries and popular queries, the SPI implementation shows good performance. |
While doing the fourth subtask, blocker was found |
Found a workaround for the blocker, currently doing the load testing. |
The load test results for the SPI implementation and the jayway implementation https://docs.google.com/spreadsheets/d/1wvb6a11Oh7oZkjKAkEeDU9pFpmrCFVpsNyTQu5_-pHk/edit?usp=sharing According to this the average response time and the throughput of the SPI implementation is better in below ways. I used |
Important to note that, SPI implementation is 20% slower for jsonpath expressions like |
Now, as the next step, I am Implementing unit tests for the SPI implementation. |
A small suggestion, instead of sharing the google doc link which is restricted to few people, shall we add screenshot or doc link that can be accessed by everyone? |
+1, will share a public doc |
PR for the implementation |
Issue found for the |
Currently JsonPath is not supported for Ballerina json types.
The aim of this task is to introduce the Jsonpath support for Ballerina Json type.
The text was updated successfully, but these errors were encountered: