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

ACS-7587 Implement v1 Bulk API to add items to a hold #2656

Conversation

damianujma
Copy link
Contributor

@damianujma damianujma commented May 17, 2024

Added the following properties:

  • rm.hold.bulk.threadCount (default: 2) - The number of worker threads.
  • rm.hold.bulk.maxItems (default: 1000) - The maximum number of total items to process in a single bulk operation.
  • rm.hold.bulk.logging.interval (default: 1000) - The number of entries to process before reporting progress.
  • rm.hold.bulk.batchSize (default: 100) - The number of entries to be fetched from the Search Service in the BatchProcessWorkProvider.getNextWork() method.
  • rm.hold.bulk.itemsPerTransaction (default: 1) - The number of entries we process at a time in a transaction. If the umber is 1, the advantage of this is that when a transaction fails, correctly added entries are not rollbacked. If the number of the items per transaction is e.g. 100 then if one entry fails in a transaction, 99 entries are rollbacked and the BatchMonitor returns only 1 error and 99 successfully processed entries. It's worth thinking.

Note: Support for the clustered environment will be added in the Enterprise Repo.

Copy link
Member

@tpage-alfresco tpage-alfresco left a comment

Choose a reason for hiding this comment

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

LGTM

@damianujma damianujma requested a review from dsibilio May 27, 2024 13:53
@damianujma damianujma merged commit 3cda821 into feature/ACS-7556_bulk_update_in_legal_holds May 28, 2024
87 checks passed
@damianujma damianujma deleted the feature/ACS-7587_implement_v1_bulk_api_to_add_items branch May 28, 2024 10:28
@damianujma damianujma restored the feature/ACS-7587_implement_v1_bulk_api_to_add_items branch May 28, 2024 10:28
damianujma added a commit that referenced this pull request Jun 19, 2024
* ACS-7557 Add Legal Holds Bulk v1 API (#2624)

* ACS-7557 Add Legal Holds Bulk v1 API

* ACS-7557 Improve v1 API

* ACS-7557 Replace processId with bulkStatusId

* ACS-7587 Implement v1 Bulk API to add items to a hold (#2656)

* ACS-7557 Add bulk API design

* ACS-7557 Fix 

* ACS-7557 Add permissions checks 

* ACS-7557 Add IT tests

* ACS-7557 Add comments + logging

* ACS-7557 Refactor

* ACS-7557 Reimplement task container

* ACS-7557 Refactor code

* ACS-7587 Remove merge leftovers

* ACS-7587 Refactor 

* ACS-7587 Tests 

* ACS-7587 Change DefaultHoldBulkMonitor

* ACS-7587 Reimplement BulkStatusUpdater

* ACS-7587 Fix PMD issues

* ACS-7587 Fix PMD isues

* ACS-7587 Refactor

* ACS-7587 Add test files alternately

* ACS-7587 Refactor code

* ACS-7587 Improve search query

* ACS-7587 Fix PMD issues

* ACS-7587 Fix PMD issue

* ACS-7587 Fix intermittent failure

* ACS7587 Fix intermittent failure (#2681)

* ACS-7587 Implement bulk cancellations (#2683)
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.

None yet

5 participants