Goboolean / fetch-system.IaC

0 stars 0 forks source link

Feat: Implement influx client #61

Closed lsjtop10 closed 3 days ago

lsjtop10 commented 1 week ago

변경 설명

InfluxDB에서 사용할 수 있는 쿼리를 추상화했습니다. 그런데 초안이라서 수정이 필요할 것 같습니다. 보시고 의견 주세요 :)

mulmuri commented 1 week ago

여기서는 굳이 mapper 안써도 되지 않나요?

lsjtop10 commented 1 week ago

그러면 매퍼 쓰지 말고 여기서는 Influx 포인트 타입으로 받을까요?

mulmuri commented 1 week ago

네네 여기는 타입을 엄격하게 관리하는 쪽으로 합시다!

lsjtop10 commented 1 week ago

네네 여기는 타입을 엄격하게 관리하는 쪽으로 합시다!

그러면 mapper를 internal package로 하면 안 되고 여기 쿼리도 손 볼게 있네요.

lsjtop10 commented 1 week ago

Ask: 테스트 코드 작성중인데 질문 있습니다. Bucket 구성은 인프라단에서 완벽히 세팅된다고 간주할 수 있나요?

mulmuri commented 1 week ago

Bucket 구성이 인프라단에서 완벽히 세팅된다는게 어떤 의미인가요? 초기 상태에는 버킷만 존재하고 내부에 데이터는 없습니다. 필요하다면 테스트의 Arrange 단계에서 직접 넣어야 해요

mulmuri commented 1 week ago

코드가 너무 길어진다 싶으면 테스트코드랑 일반 코드랑 PR 분리해주세요 588줄은.. 너무 길어요....

lsjtop10 commented 1 week ago

코드가 너무 길어진다 싶으면 테스트코드랑 일반 코드랑 PR 분리해주세요 588줄은.. 너무 길어요....

아 이거 저번에 merge된 mapper 이동하면서 이렇게 늘어났네요. 이미 merge된 거라 세심하지 못했습니다. 꼭 조심하겠습니다.

lsjtop10 commented 1 week ago

Bucket 구성이 인프라단에서 완벽히 세팅된다는게 어떤 의미인가요? 초기 상태에는 버킷만 존재하고 내부에 데이터는 없습니다. 필요하다면 테스트의 Arrange 단계에서 직접 넣어야 해요

저희가 Bucket을 여러 개 쓰기로 했습니다. 그래서 도커를 이용해 InfluxDB를 띄울 때 초기화 과정에서 bucket이 사용할 만큼 생성되는지 여쭤본 것이었습니다. 아니라면 Bucket생성까지 Arrange단계에서 해야 하기 때문입니다.
아니면 Bucket을 생성하는 코드도 포함해야 하기 때문입니다.

mulmuri commented 1 week ago

일단 버킷 여러개 생성하는건 가능합니다. 다만 테스트 용도면 버킷 하나만 있어도 되지 않나요? 결국 버킷 자체도 환경변수로 관리하게 될텐데요..! 단위테스트에서 버킷은 그냥 test 이런걸로 해도 됩니다

mulmuri commented 1 week ago

코드가 너무 길어진다 싶으면 테스트코드랑 일반 코드랑 PR 분리해주세요 588줄은.. 너무 길어요....

아 이거 저번에 merge된 mapper 이동하면서 이렇게 늘어났네요. 이미 merge된 거라 세심하지 못했습니다. 꼭 조심하겠습니다.

이 브랜치에서 메인 머지하고 깃헙에 푸쉬하면 사라질거에요

lsjtop10 commented 1 week ago

여기서 다른 것까지 구현했다간 PR이 너무 커지겠군요

lsjtop10 commented 6 days ago

size가 200줄이 넘을 것 같아서 테스트는 분리하는 것이 좋겠습니다.