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
chore: store value perf #19247
chore: store value perf #19247
Conversation
/ok-to-test sha=a0306cb |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3786856974. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
/ok-to-test sha=1465f5e |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3787845570. |
The following are new failures, please fix them before merging the PR cypress/integration/Smoke_TestSuite/ClientSideTests/ActionExecution/ClearStore_spec.ts |
/ok-to-test sha=7a9df75 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3791572901. |
/ok-to-test sha=6fb0a11 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3872530103. |
/ok-to-test sha=c1f2ef6 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3872758699. |
/ok-to-test sha=8a5d9d5 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3872882804. |
The following are new failures, please fix them before merging the PR cypress/integration/Smoke_TestSuite/ClientSideTests/BugTests/DatasourceSchema_spec.ts |
The following are new failures, please fix them before merging the PR cypress/integration/Smoke_TestSuite/ClientSideTests/Git/GitDiscardChange/DiscardChanges_spec.js |
/ok-to-test sha=c2900d5 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3876313743. |
The following are new failures, please fix them before merging the PR cypress/integration/Smoke_TestSuite/ServerSideTests/JsFunctionExecution/JSFunctionExecution_spec.ts |
/ok-to-test sha=8f9c4e1 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3879861304. |
The following are new failures, please fix them before merging the PR cypress/integration/Smoke_TestSuite/ClientSideTests/BugTests/DatasourceSchema_spec.ts |
Description
Batch store ops (storeValue, removeValue and clearStore) requests for performance.
Existing implementation of these store functions make a request to the main thread per invocation.
All occurrences of store functions are now batched and sent together.
Code changes.
Fixes #14760
Type of change
How Has This Been Tested?
Test Plan
Issues raised during DP testing
Checklist:
Dev activity
QA activity: