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

NotificationIndex Content Column Size is too high #13934

Closed
jtkech opened this issue Jul 3, 2023 · 0 comments · Fixed by #13935
Closed

NotificationIndex Content Column Size is too high #13934

jtkech opened this issue Jul 3, 2023 · 0 comments · Fixed by #13935
Labels
bug 🐛
Milestone
1.7

Comments

@jtkech
Copy link
Member

jtkech commented Jul 3, 2023 •
edited
Loading

With MySql in a given inner index the total of the included columns sizes should not be greater than 3072 bytes, so with the utf8mb4_general_ci not greater than 3072 / 4 = 768 chars. We already fixed the same issue for the Alias column size that was a little too high, see #13585 (comment).

So at least for MySql, NotificationIndexContentLength which is equal to 2500 is too high.

I will suggest a PR.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug 🐛
Projects
None yet
Milestone
1.7
Development

Successfully merging a pull request may close this issue.

NotificationIndex Content Column Size is too high OrchardCMS/OrchardCore
2 participants
@sebastienros @jtkech