Skip to content

[MNT-25563] Removing a document from a Records hold Optimization#3859

Open
ShreasiK wants to merge 1 commit intomasterfrom
fix/MNT-25563_Removing-document-from-hold-optimization
Open

[MNT-25563] Removing a document from a Records hold Optimization#3859
ShreasiK wants to merge 1 commit intomasterfrom
fix/MNT-25563_Removing-document-from-hold-optimization

Conversation

@ShreasiK
Copy link
Contributor

@ShreasiK ShreasiK commented Feb 23, 2026

  • Fix slow removal of records from holds containing many children (e.g., 30k records) by replacing child lookup with parent lookup

  • Replace getHeld(hold).contains(nodeRef) with getParentHolds(nodeRef).contains(hold) in both removeFromHolds() and addToHolds() to avoid triggering huge number of after-invocation permission checks via getChildAssocs

For more details - https://hyland.atlassian.net/browse/MNT-25563

@github-actions
Copy link



Scan Summary:
PIPELINE_SCAN_VERSION: 26.2.0-0
STATIC_ENGINE_VERSION: 20260206231001
DEV-STAGE: DEVELOPMENT
PROJECT-NAME: alfresco-community-repo
SCAN_ID: 99c7133a-5109-4883-abb0-85340db14df9
SCAN_STATUS: SUCCESS
SCAN_MESSAGE: Scan successful. Results size: 3575 bytes
====================
Analysis Successful.
====================

==========================
Found 2 Scannable modules.
==========================
reduced.war
JS files within reduced.war

===================
Analyzed 2 modules.
===================
reduced.war
JS files within reduced.war

==================
Analyzed 2 issues.
==================

Total flaws found: 2, New flaws found: 0 as compared to baseline


==================================
SUCCESS: No issues passed filters.
==================================

[23 Feb 2026 14:17:12,1650] PIPELINE-SCAN INFO: Writing Scan Summary to file '/home/runner/work/alfresco-community-repo/alfresco-community-repo/results.json'.

@ShreasiK ShreasiK requested a review from bsayan2 February 25, 2026 07:54
Copy link
Contributor

@bsayan2 bsayan2 left a comment

Choose a reason for hiding this comment

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

LGTM

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.

2 participants