Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

New DB-Limit on String Length #1152

Closed
jotoeri opened this issue Apr 1, 2022 · 1 comment · Fixed by #1149
Closed

New DB-Limit on String Length #1152

jotoeri opened this issue Apr 1, 2022 · 1 comment · Fixed by #1149
Labels
3. to review Waiting for reviews bug Something isn't working high High priority integration Compatibility with other apps/systems
Milestone
2.5

Comments

@jotoeri
Copy link
Member

jotoeri commented Apr 1, 2022

Ref. nextcloud/server#31679
Unfortunately we intentionally set the String-Column to 8192, as we had the use-case of 4900 chars on the form-description (#529 (comment)). I wouldn't like to strip that now again... 😢

@nickvergessen Can you give me a hint on how to properly walk around this limit now on forms? Is maybe MAX_STRING_SIZE initialization parameter is set to EXTENDED given, so we could move to a VARCHAR2 Type?

VARCHAR2 Maximum size: 4000 bytes, or 32767 bytes if the MAX_STRING_SIZE initialization parameter is set to EXTENDED

Ref. Oracle Docs
@jotoeri jotoeri added bug Something isn't working 1. to develop Accepted and waiting to be taken care of integration Compatibility with other apps/systems high High priority labels Apr 1, 2022
@jotoeri jotoeri added this to the 3.0 milestone Apr 1, 2022
@nickvergessen
Copy link
Member

nickvergessen commented Apr 1, 2022 •
edited
Loading

Instead of String use Text and that can be up to 65k chars:
https://github.com/nextcloud/server/blob/master/core/Migrations/Version13000Date20170718121200.php#L835-L837

@jotoeri jotoeri mentioned this issue Apr 2, 2022
Merged
@jotoeri jotoeri added 3. to review Waiting for reviews and removed 1. to develop Accepted and waiting to be taken care of labels Apr 2, 2022
@tcitworld tcitworld mentioned this issue Apr 2, 2022
Merged
@jotoeri jotoeri linked a pull request Apr 2, 2022 that will close this issue
Move to NC-Internal Db-Types #1149
Merged
@jotoeri jotoeri modified the milestones: 3.0, 2.5 Apr 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
3. to review Waiting for reviews bug Something isn't working high High priority integration Compatibility with other apps/systems
Projects
None yet
Milestone
2.5
Development

Successfully merging a pull request may close this issue.

Move to NC-Internal Db-Types nextcloud/forms
2 participants
@nickvergessen @jotoeri