Skip to content

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Mar 19, 2025

Summary

If Oracle is not explicitly disabled by an app we check the table names / key names / column names for legacy Oracle length of max. 30 characters.
But since Oracle 12c the limit was increased to 128 characters.

So nowadays we are limited by MySQL / Mariadb and Postgres:

  • MySQL / MariaDB: 64 characters
  • Postgres: 63 characters
  • SQLite: unlimited
  • Oracle: 128 characters

This PR adjusts the checks to only check for Oracle specific behavior but limit the name length to 63.

Checklist

susnux added 4 commits March 19, 2025 23:26
We do not support Oracle 11 anymore but at least Oracle 12c (12.2).
So the limitation is gone (Oracle now supports up to 128 character long
names).

Instead we are now limited by MySQL (64 characters) and PostgreSQL (63
characters).

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux added 2. developing Work in progress pending documentation This pull request needs an associated documentation update feature: database Database related DB labels Mar 19, 2025
@susnux susnux added this to the Nextcloud 32 milestone Mar 19, 2025
@susnux susnux force-pushed the chore/support-longer-names branch 4 times, most recently from 1d770b0 to 98d731d Compare March 20, 2025 08:57
@susnux susnux marked this pull request as ready for review March 20, 2025 08:58
@susnux susnux requested a review from a team as a code owner March 20, 2025 08:58
@susnux susnux requested review from artonge, nickvergessen, provokateurin and yemkareems and removed request for a team March 20, 2025 08:58
@susnux susnux added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Mar 20, 2025
Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

11g is still supported and that is actually what all apps tests against atm iirc.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
This was referenced Aug 22, 2025
This was referenced Sep 2, 2025
This was referenced Sep 25, 2025
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
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 feature: database Database related DB pending documentation This pull request needs an associated documentation update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants