Fix old type change that was incorrect. #5
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Old repofile.content change from LONGTEXT to TEXT was too restrictive, resulting in failures in testing when existing seed data was too large. MEDIUMTEXT is more reasonable then LONGTEXT while still allowing quite large sizes. This change has been applied twice so that users out there that had applied the older change can stay in-sync.
|
Recently our team found a possibly related issue to this change. In order to make this modification to the database, I believe that there are 'diffjson' columns scattered throughout many tables that need to be altered as well. We found an issue when you go to apply a security group to a file that has a large size. It will not apply the security group due to the 'diffjson' column being too small and give an unrelated error message of something like " There are keys already related to this key ". We updated every diffjson column from TEXT to MEDIUMTEXT in order to get by this issue. |
|
Any update on this ? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Old repofile.content change from LONGTEXT to TEXT was too
restrictive, resulting in failures in testing when existing
seed data was too large. MEDIUMTEXT is more reasonable
then LONGTEXT while still allowing quite large sizes.
This change has been applied twice so that users out
there that had applied the older change can stay in-sync.