Skip to content

Improve performance when dealing with large numbers of shares #10724

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.

Sign up for GitHub

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

Merged
MorrisJobke merged 11 commits into from
Aug 24, 2018
Merged

Improve performance when dealing with large numbers of shares #10724

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
9598181
remove double loop for detecting changed mounts
icewind1991 Aug 16, 2018
f8116ad
use more efficient method to find mountpoint for path
icewind1991 Aug 16, 2018
f7ae235
cache OC\Files\Mount\Manager::findIn results
icewind1991 Aug 16, 2018
3155c1b
more efficient unique share target generation
icewind1991 Aug 16, 2018
c300516
more efficient way to detect added and removed mounts
icewind1991 Aug 16, 2018
bc4f10a
more efficient way to detect added and removed mounts
icewind1991 Aug 16, 2018
b79f338
re-use view instances for shared storages
icewind1991 Aug 16, 2018
b5217b6
don't check if target dir exists when using the default share target …
icewind1991 Aug 16, 2018
720b27d
only determine is sharing is disabled for a user once
icewind1991 Aug 16, 2018
4c0cbdf
tokens can't be valid local user names
icewind1991 Aug 16, 2018
67f3d6f
cache parent exists status during share setup
icewind1991 Aug 16, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
remove double loop for detecting changed mounts 
Signed-off-by: Robin Appelman <[email protected]>
  • Loading branch information
@icewind1991
icewind1991 committed Aug 16, 2018
commit 95981810c05100f0439afe6d99973e0f1bd13d39
21 changes: 12 additions & 9 deletions lib/private/Files/Config/UserMountCache.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -135,16 +135,19 @@ public function registerMounts(IUser $user, array $mounts) {
* @return ICachedMountInfo[]
*/
private function findChangedMounts(array $newMounts, array $cachedMounts) {
$new = [];
foreach ($newMounts as $mount) {
$new[$mount->getRootId()] = $mount;
}
$changed = [];
foreach ($newMounts as $newMount) {
foreach ($cachedMounts as $cachedMount) {
foreach ($cachedMounts as $cachedMount) {
$rootId = $cachedMount->getRootId();
if (isset($new[$rootId])) {
$newMount = $new[$rootId];
if (
$newMount->getRootId() === $cachedMount->getRootId() &&
(
$newMount->getMountPoint() !== $cachedMount->getMountPoint() ||
$newMount->getStorageId() !== $cachedMount->getStorageId() ||
$newMount->getMountId() !== $cachedMount->getMountId()
)
$newMount->getMountPoint() !== $cachedMount->getMountPoint() ||
$newMount->getStorageId() !== $cachedMount->getStorageId() ||
$newMount->getMountId() !== $cachedMount->getMountId()
) {
$changed[] = $newMount;
}
Expand Down Expand Up @@ -197,7 +200,7 @@ private function dbRowToMountInfo(array $row) {
}
$mount_id = $row['mount_id'];
if (!is_null($mount_id)) {
$mount_id = (int) $mount_id;
$mount_id = (int)$mount_id;
}
return new CachedMountInfo($user, (int)$row['storage_id'], (int)$row['root_id'], $row['mount_point'], $mount_id, isset($row['path']) ? $row['path'] : '');
}
Expand Down