Skip to content

Conversation

@iamjooon2
Copy link
Member

@iamjooon2 iamjooon2 commented Mar 1, 2025

  • common 패키지 만들어서 공용으로 쓸 페이징 구조체 하나 만들어뒀어요 (common/techbloghub.paging.go)
  • 복합 인덱스 적용
    • 발행 일자 기준으로 정렬 및 페이징 처리를 위해 복합 인덱스를 맹글어놨습니당(운영디비 반영 완료)
CREATE INDEX idx_posting_published_at_id ON postings (published_at DESC, id DESC);
  • handler를 슬슬 분리해야할 타이밍 같아요
    • 일단 시간이 늦었으니 나이브하게 올려뒀습니다 ㅎㅎ

@github-actions
Copy link

github-actions bot commented Mar 1, 2025

report: Run #62

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Pending ⏳ Other ❓ Flaky 🍂 Duration ⏱️
2 2 0 0 0 0 0 103ms

🎉 All tests passed!

Github Test Reporter

@github-actions
Copy link

github-actions bot commented Mar 1, 2025

report: Run #63

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Pending ⏳ Other ❓ Flaky 🍂 Duration ⏱️
2 2 0 0 0 0 0 544ms

🎉 All tests passed!

Github Test Reporter

@github-actions
Copy link

github-actions bot commented Mar 2, 2025

report: Run #64

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Pending ⏳ Other ❓ Flaky 🍂 Duration ⏱️
2 2 0 0 0 0 0 598ms

🎉 All tests passed!

Github Test Reporter

@iamjooon2 iamjooon2 requested a review from pilyang March 2, 2025 10:43
Copy link
Member

@pilyang pilyang left a comment

Choose a reason for hiding this comment

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

LGTM

아직 막 자세히 확인해보지는 못했는데,, 우선 빠른 연동작업을 위해 나가고 나중에 함 자세히 봐보도록 할께요!!
다음에 이어서 태그검색 api작업 되는거것죠??

@iamjooon2 iamjooon2 merged commit 672be9c into main Mar 2, 2025
2 checks passed
@iamjooon2 iamjooon2 deleted the feat/posting-search branch March 2, 2025 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants