-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[stable7] Close cursor early in calculateFolderSize #13832
Conversation
This method triggers additional queries in $this->update() so to avoid potential database locks or delays, we close the cursor as soon as it is not needed any more
let's postpone this - THX |
The inspection completed: 8 new issues |
Huh what ?! It was supposed to fail on Oracle ? |
@PVince81 Oracle is just run on master and the stable branches - not on PRs ;) |
Refer to this link for build results (access rights to CI server needed): |
@PVince81 Seems to work on Oracle:
:) |
Upgrading to 8.0.x is advised. |
This method triggers additional queries in $this->update() so to avoid
potential database locks or delays, we close the cursor as soon as it is not needed any more
This is the backport that will fail with Oracle (which we reverted here #13803)