Skip to content

Commit 6c8fab7

Browse files
authored
perf: bulk action import caching (#41807)
## Description > [!TIP] > _Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team)._ > > _Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR._ Fixes #`Issue Number` _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="@tag.ImportExport, @tag.Git" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/25814789299> > Commit: 1fc6461 > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=25814789299&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.ImportExport, @tag.Git` > Spec: > <hr>Wed, 13 May 2026 17:37:04 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Added bulk action validation support for imports with dry-run mode capability. * Enhanced import pipeline with improved datasource policy and plugin metadata handling. * **Tests** * Added validation tests for import scenarios, including public datasource policy updates and dry-run validation modes. <!-- review_stack_entry_start --> [![Review Change Stack](https://storage.googleapis.com/coderabbit_public_assets/review-stack-in-coderabbit-ui.svg)](https://app.coderabbit.ai/change-stack/appsmithorg/appsmith/pull/41807) <!-- review_stack_entry_end --> <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent d75495a commit 6c8fab7

4 files changed

Lines changed: 582 additions & 71 deletions

File tree

app/server/appsmith-server/src/main/java/com/appsmith/server/newactions/base/NewActionServiceCE.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,15 @@ public interface NewActionServiceCE extends CrudService<NewAction, String> {
3636

3737
Mono<NewAction> validateAction(NewAction newAction, boolean isDryOps);
3838

39+
Mono<Tuple2<List<NewAction>, List<NewAction>>> validateActionsBeforeImport(
40+
String artifactId, List<NewAction> freshActions, List<NewAction> existingActions);
41+
3942
Mono<Void> bulkValidateAndInsertActionInRepository(List<NewAction> newActionList);
4043

4144
Mono<Void> bulkValidateAndUpdateActionInRepository(List<NewAction> newActionList);
4245

46+
Mono<Void> bulkInsertActions(List<NewAction> newActionList);
47+
4348
/**
4449
* General purpose bulk update method that directly saves actions to the database without validation.
4550
* This method is optimized for scenarios where the actions are already validated or when

0 commit comments

Comments
 (0)