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.

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

보상 정산 방식 리팩토링 #60

Open
wants to merge 20 commits into
base: main
Choose a base branch
from
Open

보상 정산 방식 리팩토링 #60

wants to merge 20 commits into from

Conversation

ipdae
Copy link
Member

@ipdae ipdae commented Sep 20, 2024

  • raw bulk insert sql을 통해 성능 향상
  • tx 서명에 headless 의존성을 제거하고 직접 서명
  • 기존 정산방식에서 전체목록을 직접 생성하던 방식대신 스케쥴러를 통해 DB에 Transaction, WorldBoss, WorldBossRewardAmount 모델을 정해진 크기만큼 나눠서 생성하고 모든 모델이 준비되면 슬랙에 csv로 결과를 업로드하도록 변경

@ipdae ipdae requested a review from U-lis September 20, 2024 09:12
@ipdae ipdae self-assigned this Sep 20, 2024
],
]
)
if memo is not None:
Copy link

Choose a reason for hiding this comment

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

The function parameter says memo is mandatory.
If memo is really nullable, the function parameter can be nullable like memo: Optional[str] = None

world_boss/app/raid.py Outdated Show resolved Hide resolved
@ipdae
Copy link
Member Author

ipdae commented Sep 24, 2024

임시배포 이후 중간 순위가 비는 현상이 발생. 조사 후 추가 수정해서 다시 리뷰요청하겠습니다.

@ipdae
Copy link
Member Author

ipdae commented Oct 2, 2024

아래와 같은 처리들이 추가됐습니다. 리뷰 부탁드립니다. @U-lis

  • 월드보스 순위정산 로직상 동일 순위가 발생할 가능성이 생겨서 ranking 필드의 중복허용
  • 태스크 중간과정에 실패할 경우 Tx모델만 미리 생성되고 보상정산이 안되어 논스가 계속해서 높아지는 문제가 발생하여 네트워크통신 종료후 관련 모델을 한 메서드 내부에서 처리하도록 설정

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Review
Development

Successfully merging this pull request may close these issues.

2 participants