Skip to content

Full repository exports #11521#11678

Merged
vbradnitski merged 8 commits intomasterfrom
issue-11521
Jan 19, 2026
Merged

Full repository exports #11521#11678
vbradnitski merged 8 commits intomasterfrom
issue-11521

Conversation

@vbradnitski
Copy link
Copy Markdown
Contributor

No description provided.

@vbradnitski vbradnitski requested a review from Copilot January 16, 2026 15:23
@vbradnitski vbradnitski linked an issue Jan 16, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for configurable batch size in repository export functionality, allowing users to control the number of nodes exported in each batch for better performance optimization.

Changes:

  • Added batchSize parameter to export APIs (REST, JavaScript library, and core export service)
  • Modified export implementation to process nodes in batches using the specified batch size
  • Refactored node export logic to use a query-based approach instead of recursive parent-child traversal

Reviewed changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
export_params_with_batch_size.json Test resource with batch size parameter
ExportRunnableTaskTest.java Added tests for batch size functionality
RepositoryResourceTest.java Added test for REST endpoint with batch size
ExportRunnableTask.java Added batch size field and builder method
ExportNodesRequestJson.java Added batch size parameter to request model
RepositoryResource.java Passes batch size to export task
ExportHandlerTest.java Added test for JavaScript library batch size
export.ts Added batch size to TypeScript interface and documentation
exportNodesWithBatchSize.js Example showing batch size usage
ExportHandler.java Added batch size setter method
NodeExporter.java Core implementation of batched export logic
ExportServiceImpl.java Passes batch size to NodeExporter
ExportNodesParamsTest.java Tests for batch size parameter
ApplyVersionAttributesParams.java Removed empty JavaDoc line
ExportNodesParams.java Added batch size field with default value of 1000

vbradnitski and others added 2 commits January 16, 2026 16:25
…tNodesParams.java

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@vbradnitski vbradnitski requested a review from rymsha January 19, 2026 08:32
@vbradnitski vbradnitski marked this pull request as ready for review January 19, 2026 08:32
# Conflicts:
#	modules/core/core-api/src/main/java/com/enonic/xp/node/ApplyVersionAttributesParams.java
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 19, 2026

Codecov Report

❌ Patch coverage is 86.95652% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.66%. Comparing base (d238cc6) to head (d07a632).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...a/com/enonic/xp/core/impl/export/NodeExporter.java 81.81% 4 Missing and 4 partials ⚠️
.../enonic/xp/core/impl/export/ExportServiceImpl.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             master   #11678   +/-   ##
=========================================
  Coverage     86.66%   86.66%           
- Complexity    19714    19721    +7     
=========================================
  Files          2538     2538           
  Lines         66802    66835   +33     
  Branches       5363     5368    +5     
=========================================
+ Hits          57894    57924   +30     
  Misses         6317     6317           
- Partials       2591     2594    +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vbradnitski vbradnitski merged commit e61f53a into master Jan 19, 2026
10 of 11 checks passed
@vbradnitski vbradnitski deleted the issue-11521 branch January 19, 2026 11:56
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

Successfully merging this pull request may close these issues.

Full repository exports

3 participants