Skip to content

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

Open
provokateurin wants to merge 1 commit intomasterfrom
fix/db/postgres-filecache-path-prefix-index
Open

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

Conversation

@provokateurin
Copy link
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
Member Author

/backport to stable33

@provokateurin
Copy link
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants