Skip to content
This repository has been archived by the owner on May 19, 2024. It is now read-only.

[BM-195] Bidding Fk Fetch Type 변경(Lazy -> EAGER) #73

Merged
merged 1 commit into from
Aug 3, 2022
Merged

Conversation

dojinyou
Copy link
Member

@dojinyou dojinyou commented Aug 3, 2022

주요 내용

  • Bidding Entity의 Fetch Type을 Lazy에서 Eager로 변경
    • 비딩 금액만으로는 무언가를 표현할 수 없어서 상품과 비딩한 유저의 정보가 늘 필요하다고 생각해서 변경하였습니다.

Copy link
Member

@UJ15 UJ15 left a comment

Choose a reason for hiding this comment

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

👍

@github-actions
Copy link

github-actions bot commented Aug 3, 2022

📊 BidMarket-pr-73 분석 결과 확인 링크

@codecov-commenter
Copy link

Codecov Report

Merging #73 (0ac2955) into main (138a563) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##               main      #73   +/-   ##
=========================================
  Coverage     76.95%   76.95%           
  Complexity      237      237           
=========================================
  Files            52       52           
  Lines           794      794           
  Branches         24       24           
=========================================
  Hits            611      611           
  Misses          172      172           
  Partials         11       11           
Impacted Files Coverage Δ
...va/com/saiko/bidmarket/bidding/entity/Bidding.java 71.42% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 138a563...0ac2955. Read the comment docs.

@dojinyou dojinyou merged commit d1f193d into main Aug 3, 2022
@dojinyou dojinyou deleted the BM-195 branch August 3, 2022 10:52
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants