fix(Challenge): Remove @Lob annotations from Challenge entity fields #84
Conversation
Removed @lob annotations from blurb, promptMd, and expectedAnswer fields.
ArchILLtect
added
area:persistence
There was a problem hiding this comment.
Pull request overview
This pull request simplifies the database mapping configuration for the Challenge entity by removing redundant @Lob annotations from TEXT fields that already have explicit column definitions. The @Lob annotations on blurb, promptMd, and expectedAnswer fields are removed since these fields already specify columnDefinition = "TEXT", making the @lob annotation unnecessary for PostgreSQL.
Changes:
- Removed
@Lobannotations from three Challenge entity fields while retaining their explicit TEXT column definitions - Simplified JPA mapping configuration by eliminating redundant annotations
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Removed @lob annotation from the code field in Submission class.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
github-project-automation
bot
moved this from Todo
to Done
in CodeForge – Post MVP Release
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
1 participant
This pull request makes minor adjustments to how large text fields are mapped in the entity classes. Specifically, it removes the unnecessary
@Lobannotation from several fields that are already mapped to database columns with theTEXTtype.@Lobannotation from theblurb,promptMd, andexpectedAnswerfields in theChallengeentity, since these fields are already mapped asTEXTcolumns. [1] [2]@Lobannotation from thecodefield in theSubmissionentity, as it is also already mapped for large text storage.