Skip to content

dont delegate Mount\Manager::getByNumericId to getByStorageId - #36566

Merged
juliusknorr merged 4 commits into
masterfrom
mount-find-by-numeric-id
Feb 8, 2023
Merged

dont delegate Mount\Manager::getByNumericId to getByStorageId#36566
juliusknorr merged 4 commits into
masterfrom
mount-find-by-numeric-id

Conversation

@icewind1991

Copy link
Copy Markdown
Member

Should be faster than getting the string storage id

Also includes a bonus cleanup of getByStorageId

Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991icewind1991 added the 3. to review Waiting for reviews label Feb 6, 2023
@icewind1991icewind1991 added this to the Nextcloud 26 milestone Feb 6, 2023
@icewind1991
icewind1991 requested review from a team, ArtificialOwl, blizzz and come-nc and removed request for a teamFebruary 6, 2023 17:21
@icewind1991
icewind1991force-pushed the mount-find-by-numeric-id branch 2 times, most recently from 0cb467f to cd04615CompareFebruary 7, 2023 12:51
Comment threadlib/private/Files/Mount/MountPoint.php Fixed
Comment threadlib/private/Files/Mount/MountPoint.php Fixed
@icewind1991
icewind1991force-pushed the mount-find-by-numeric-id branch from cd04615 to 7fd96c2CompareFebruary 7, 2023 13:28
Comment threadlib/private/Files/Mount/MountPoint.php Fixed
Signed-off-by: Robin Appelman <robin@icewind.nl>
@juliusknorr
juliusknorr merged commit 08802ad into masterFeb 8, 2023
@juliusknorr
juliusknorr deleted the mount-find-by-numeric-id branch February 8, 2023 19:04
@blizzzblizzz mentioned this pull request Feb 9, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsperformance 🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@icewind1991@juliusknorr@github-advanced-security@come-nc