Skip to content

fix(DB): Ignore intentionally missing fs_storage_path_prefix index on PostgreSQL#58734

Merged
provokateurin merged 1 commit intomasterfrom
fix/db/postgres-filecache-path-prefix-index
Apr 20, 2026
Merged

fix(DB): Ignore intentionally missing fs_storage_path_prefix index on PostgreSQL#58734
provokateurin merged 1 commit intomasterfrom
fix/db/postgres-filecache-path-prefix-index

Conversation

@provokateurin
Copy link
Copy Markdown
Member

The index is not created on PostgreSQL in the migration:

if ($this->connection->getDatabaseProvider() !== IDBConnection::PLATFORM_POSTGRES) {
$table->addIndex(['storage', 'path'], 'fs_storage_path_prefix', [], ['lengths' => [null, 64]]);
}

Any new instance with PostgreSQL shows a warning for this missing index.
When the index was added in #28541 it was correctly excluded, but the condition got lost in some refactoring.

@provokateurin provokateurin added this to the Nextcloud 34 milestone Mar 5, 2026
@provokateurin provokateurin requested a review from a team as a code owner March 5, 2026 09:28
@provokateurin provokateurin removed the request for review from a team March 5, 2026 09:28
@provokateurin provokateurin added the 3. to review Waiting for reviews label Mar 5, 2026
@provokateurin
Copy link
Copy Markdown
Member Author

/backport to stable33

@provokateurin
Copy link
Copy Markdown
Member Author

/backport to stable32

@provokateurin provokateurin force-pushed the fix/db/postgres-filecache-path-prefix-index branch from 5728f78 to 0e89319 Compare March 5, 2026 09:32
… PostgreSQL

Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin provokateurin force-pushed the fix/db/postgres-filecache-path-prefix-index branch from 0e89319 to 8ae9424 Compare March 5, 2026 09:43
@provokateurin provokateurin enabled auto-merge March 5, 2026 10:45
@provokateurin provokateurin merged commit b23ab0c into master Apr 20, 2026
185 of 187 checks passed
@provokateurin provokateurin deleted the fix/db/postgres-filecache-path-prefix-index branch April 20, 2026 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants