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

Fix memory usage on integrity check #6660

Merged
merged 1 commit into from
Nov 16, 2023

Conversation

yurabakhtin
Copy link
Contributor

What kind of change does this PR introduce? (check at least one)

  • Bugfix

The PR fulfills these requirements:

  • All tests are passing
  • Changelog was modified

Other information:
Fix similar issue as humhub/cfiles#198

@yurabakhtin yurabakhtin requested a review from luke- November 15, 2023 14:37
Copy link

what-the-diff bot commented Nov 15, 2023

PR Summary

  • Occasional High Memory Usage Resolved
    A fix has been implemented to address the issue of high memory consumption when running integrity checks.
  • Efficient File Deletion Mechanism
    Changes have been made in the way files are deleted to ensure the process runs smoothly even if the target file doesn't exist. The code now iterates over each file individually.
  • Improved Post Deletion Function
    Just as with files, deleting a post when its content record doesn't exist will no longer cause problems. The system now efficiently handles each post in isolation during the deletion process.

@luke- luke- merged commit 5cdb26c into master Nov 16, 2023
6 checks passed
@luke- luke- deleted the fix/198-integrity-check-memory-usage branch November 25, 2023 18:48
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