Skip to content

don't always check if we need to setup the object store root - #38415

Merged
blizzz merged 1 commit into
masterfrom
object-store-init-root
May 24, 2023
Merged

don't always check if we need to setup the object store root#38415
blizzz merged 1 commit into
masterfrom
object-store-init-root

Conversation

@icewind1991

Copy link
Copy Markdown
Member

This prevents having to check if the root is initialized every time we construct an object store.

Normally, the impact of this is fairly small, but when objectstore.multibucket.preview-distribution is enbled, we always setup 256 object stores

@icewind1991icewind1991 added the 3. to review Waiting for reviews label May 23, 2023
@icewind1991icewind1991 added this to the Nextcloud 28 milestone May 23, 2023
@icewind1991
icewind1991 requested review from a team, ArtificialOwl, blizzz and nfebe and removed request for a teamMay 23, 2023 12:38
@icewind1991icewind1991 added 2. developing Work in progress and removed 3. to review Waiting for reviews labels May 23, 2023
@icewind1991
icewind1991force-pushed the object-store-init-root branch 3 times, most recently from 9e9dc14 to 2f05d0fCompareMay 23, 2023 15:02
Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991
icewind1991force-pushed the object-store-init-root branch from 2f05d0f to cc0ad25CompareMay 23, 2023 15:44
@icewind1991icewind1991 added 3. to review Waiting for reviews and removed 2. developing Work in progress labels May 23, 2023
@blizzz
blizzz merged commit b2e3426 into masterMay 24, 2023
@blizzz
blizzz deleted the object-store-init-root branch May 24, 2023 11:06
@icewind1991

Copy link
Copy Markdown
MemberAuthor

/backport to stable27

@icewind1991

Copy link
Copy Markdown
MemberAuthor

/backport to stable26

@icewind1991

Copy link
Copy Markdown
MemberAuthor

/backport to stable25

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

@icewind1991@blizzz@juliusknorr