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

Feature/#332 - 누락된 최신 차트데이터가 존재하면 API 요청 후 차트데이터 전달 #344

Merged
merged 7 commits into from
Dec 3, 2024

Conversation

xjfcnfw3
Copy link
Collaborator

@xjfcnfw3 xjfcnfw3 commented Dec 3, 2024

close #332

✅ 작업 내용

  • 성능이 느려지는 painToInstance 제거
  • 누락된 차트 데이터가 있는 경우 큐에 삽입 후 요청이 처리되면 반환하도록 변경
  • 불필요한 차트 데이터 클래스제거

📌 이슈 사항

  • 만약 주식 API 요청이 많이 쌓이면 차트 데이터를 늦게 받을 수 있습니다.
  • 일단 차트 데이터는 추가된 시점의 데이터만 전달합니다. 이때 웹소켓이 연결되어 있으면 최신 라이브데이터를 받을 수 있습니다. 이걸로 차트 업데이트를 진행하면 될 거 같습니다.
  • 만약 차트가 먼가 이상하면 알려주세요!

😎 체크 사항

  • label 설정 확인
  • 브랜치 방향 확인

@xjfcnfw3 xjfcnfw3 added ✨feature 기능 개발 BE labels Dec 3, 2024
@xjfcnfw3 xjfcnfw3 requested a review from a team December 3, 2024 16:21
@swkim12345
Copy link
Collaborator

이거 혹시 모르니깐 내일 아침에 머지할게요 데이터 서버에서 다 먹음 할게요..

@@ -1,6 +1,6 @@
import { Inject, Injectable } from '@nestjs/common';
import { Cron } from '@nestjs/schedule';
import { DataSource } from 'typeorm';
import { DataSource, QueryFailedError } from 'typeorm';
Copy link
Collaborator

Choose a reason for hiding this comment

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

오? 아니 왜 이런 걸 쿡북에 안 작성해주는거야

Comment on lines +57 to +63
isSameDate(dateToCompare: NewDate | Date): boolean {
return (
this.getFullYear() === dateToCompare.getFullYear() &&
this.getMonth() === dateToCompare.getMonth() &&
this.getDate() === dateToCompare.getDate()
);
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

아 이걸 추가하셨군요!

Comment on lines +60 to +64
const queryBuilder = this.createQueryBuilder(
entity,
stockId,
manager,
lastStartTime,
Copy link
Collaborator

Choose a reason for hiding this comment

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

이..이게 뭐고... 상당히 쉽게 할 수 있네요

Copy link
Collaborator

@swkim12345 swkim12345 left a comment

Choose a reason for hiding this comment

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

전반적으로 리팩토링하셔서 이렇게도 할 수 있구나 하면서 코드를 봤네요! 수고하셨습니다!

Copy link
Collaborator

@baegyeong baegyeong left a comment

Choose a reason for hiding this comment

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

제가 잘 이해한건지 모르겠지만 기존 구조랑 비슷하게 하려고 하신 것 같아요 수고하셨습니다!!

@swkim12345 swkim12345 merged commit d8b4a8d into dev-be Dec 3, 2024
@swkim12345 swkim12345 deleted the feature/#332 branch December 5, 2024 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE ✨feature 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants