Skip to content

feat(scanner): enable use of a setting to disable DB transaction - #56383

Draft
mgallien wants to merge 1 commit into
masterfrom
bugfix/filesScanDisableTransactions
Draft

feat(scanner): enable use of a setting to disable DB transaction#56383
mgallien wants to merge 1 commit into
masterfrom
bugfix/filesScanDisableTransactions

Conversation

@mgallien

Copy link
Copy Markdown
Contributor

while doing a files scan from storage, we may want not to use DB transactions

enable the use of a setting for that

define filescanner_no_transactions in system config to enable this (i.e. in config.php)

  • Resolves: #

Summary

TODO

  • ...

Checklist

@mgallienmgallien self-assigned this Nov 12, 2025
@mgallien
mgallien requested a review from a team as a code ownerNovember 12, 2025 09:37
@mgallienmgallien added the 3. to review Waiting for reviews label Nov 12, 2025
@mgallien
mgallien requested review from Altahrim, come-nc, icewind1991 and yemkareems and removed request for a teamNovember 12, 2025 09:37
@artonge
artongeforce-pushed the bugfix/filesScanDisableTransactions branch from a38930a to 84e07c1CompareNovember 12, 2025 10:27

@artongeartonge left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Blocking, as this is only intended to be a patch.

Also, removed the unnecessary changes to keep it minimal.

@artonge
artongeforce-pushed the bugfix/filesScanDisableTransactions branch from 84e07c1 to e3d4d84CompareNovember 12, 2025 10:56
@artongeartonge closed this Nov 12, 2025
@artonge
artongeforce-pushed the bugfix/filesScanDisableTransactions branch from e3d4d84 to 8029dd6CompareNovember 12, 2025 11:17
@artongeartonge reopened this Nov 12, 2025
@artonge

Copy link
Copy Markdown
Collaborator

/backport! to stable31

while doing a files scan from storage, we may want not to use DB transactions
enable the use of a setting for that
define filescanner_no_transactions in system config to enable this (i.e. in config.php)
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
Signed-off-by: Louis Chmn <louis@chmn.me>
@artonge
artongeforce-pushed the bugfix/filesScanDisableTransactions branch from 4b11bab to f6fe536CompareNovember 12, 2025 11:31
@artonge
artonge marked this pull request as draft April 3, 2026 12:21
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@mgallien@artonge@joshtrichards