Skip to content

Conversation

@leebs0521
Copy link
Collaborator

@leebs0521 leebs0521 commented Jan 14, 2025

resolved :

📌 과제 설명

엔티티 클래스에서 @lob 붙은 컬럼에 대한 에러 수정
ddl-auto validate로 변경

image

👩‍💻 요구 사항과 구현 내용

  • @lob 붙은 어노테이션들 @Cloumn(length)로 길이 변경 해주었습니다. 임시, 더 필요한 경우 바꿔주셔도 됩니다.
    • 기관 봉사자 자기소개 500자, 쪽지 500자
    • 커뮤니티, 모집글, 리뷰 1000자
    • 커뮤니티 댓글 200자
  • ddl-auto 변경 이유
    • 이거 괜찮을지 코멘트 주시면 감사하겠습니다.
      • 운영 DB 스키마 최신화하는 것은 PR전에 하면 충분하다고 생각해서 PR 직전에 update로 바꿔서 최신화해줘도 될 것 같아요.(다시 validate로 변경 후 PR)

✅ PR 포인트 & 궁금한 점

 - 리뷰, 모집글, 커뮤니티 1000자
 - 센터, 봉사자 자기소개, 쪽지 500자
 - 댓글 200자
@leebs0521 leebs0521 self-assigned this Jan 14, 2025
@leebs0521 leebs0521 linked an issue Jan 14, 2025 that may be closed by this pull request
2 tasks
@sonarqubecloud
Copy link

@m-a-king
Copy link
Collaborator

ddl-auto 변경 이유
이거 괜찮을지 코멘트 주시면 감사하겠습니다.
운영 DB 스키마 최신화하는 것은 PR전에 하면 충분하다고 생각해서 PR 직전에 update로 바꿔서 최신화해줘도 될 것 같아요.(다시 validate로 변경 후 PR)

운영 디비 테이블 자주 변경시키면서 이런 상태를 자주 겪었는데, validate 좋은 것 같습니다.
무엇보다 update가 엔티티를 테이블에 완벽히 매핑하거나 업데이트 해주지 않는 경우를 자주 봐서, validate로 변경하는 것이 좋습니다.

Copy link
Collaborator

@m-a-king m-a-king left a comment

Choose a reason for hiding this comment

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

column에 length 지정하면 어떤 점이 달라지나요?

@leebs0521
Copy link
Collaborator Author

@m-a-king
저희 모집글이나 커뮤니티등의 테이블에서 content는 길이가 긴 문자열을 저장해야하는데 @lob 기본값이 컬럼 타입이 tinytext(255자 까지만 저장)해서 에러가 발생해서 고쳤습니다. 위 사진보면 좋을것 같아요. Data too long for column)

@m-a-king
Copy link
Collaborator

@m-a-king 저희 모집글이나 커뮤니티등의 테이블에서 content는 길이가 긴 문자열을 저장해야하는데 @lob 기본값이 컬럼 타입이 tinytext(255자 까지만 저장)해서 에러가 발생해서 고쳤습니다. 위 사진보면 좋을것 같아요. Data too long for column)

핑프행동 했네요 ㅋㅋㅋㅋ 감사합니다. 가끔 저 문제가 발생하면 따로 디비 수정했던 것 같은데 저런 문제가 있었군요!

@leebs0521 leebs0521 merged commit 5525c43 into main Jan 15, 2025
3 checks passed
@leebs0521 leebs0521 deleted the fix/300-refactor-using-lob-column branch January 15, 2025 03:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FIX] @Lob 사용시 DB에 tinytext 저장되는 문제 해결

4 participants