[FIX]: 회원 테스트 데이터 미적재 오류 수정

[HyeonBin2379]

작업 배경

• 부하테스트 진행에 필요한 회원 테스트 데이터가 적재되지 않은 오류 확인
  (초기 admin 계정 6개만 적재됨)

작업 내용

• CHECK_SQL 조건 수정

테스트 여부

• 재배포 후 회원 테스트 데이터 저장 여부 확인
  (회원 테스트 데이터 적재됨을 확인)

기타

이슈

이 PR과 연관된 이슈 번호를 작성해주세요. (이슈 없으면 생략 가능)

• This closes [FIX] 원격 배포 시 .csv 파일의 회원 데이터가 적재되지 않는 문제 수정 #58

Summary by CodeRabbit

릴리스 노트

• 개선 사항
  • 데이터 로딩 시 필요한 최소 데이터 건수 임계값을 상향하여 더욱 엄격한 데이터 유효성 검증 기준이 적용되었습니다. 사용자 정보 및 채팅 시간 범위 데이터에 대해 일관되게 적용됩니다.

[coderabbitai]

📝 Walkthrough

개요

ChatTimeCsvLoader와 UserCsvLoader의 데이터 존재 여부 확인 SQL이 10,000건 이상 레코드 기준으로 통일되었습니다. 기존 COUNT(*) ... LIMIT 1 방식에서 CASE WHEN COUNT(*) >= 10000 THEN 1 ELSE 0 END 형태로 변경되어 원격 배포 시 CSV 데이터 적재 조건을 명확히 합니다.

변경 사항

CSV 로더 데이터 임계값 통일

계층 / 파일	요약
데이터 적재 임계값 강화 src/main/java/org/pgsg/user_service/user/infrastructure/loader/ChatTimeCsvLoader.java, src/main/java/org/pgsg/user_service/user/infrastructure/loader/UserCsvLoader.java	CHECK_SQL 쿼리가 p_chat_time_range와 p_user 테이블의 레코드 수를 10,000건 이상 기준으로 확인하도록 변경되어, 데이터 적재 완료 조건을 통일합니다.

예상 코드 리뷰 노력

🎯 1 (간단함) | ⏱️ ~5분

제안 리뷰어

• loveletheart

축하 시

🐰 테스트 데이터 적재의 작은 문제,
임계값 통일로 크게 해결했네요!
만 개의 행이 무릎 꿇고,
원격 배포의 길을 밝히네요. ✨
데이터 로더, 이제 정확해요! 📊

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	PR은 #58 이슈의 목표(CsvLoader 수정을 통한 회원 테스트 데이터 적재)를 충족하기 위해 두 파일의 CHECK_SQL을 변경했습니다.
Out of Scope Changes check	✅ Passed	PR의 모든 변경사항이 #58 이슈 해결(CsvLoader의 CHECK_SQL 조건 수정)에 직접 관련되어 있습니다.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/#58-fix-csv-loader

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

🧹 Nitpick comments (1)

src/main/java/org/pgsg/user_service/user/infrastructure/loader/ChatTimeCsvLoader.java (1)

10-10: ⚡ Quick win

매직 넘버를 상수화하거나 설정 가능하게 만드는 것을 고려하세요.

임계값 10000이 두 로더에서 반복 사용되고 있습니다. 이를 공통 상수로 추출하거나 설정 파일에서 읽어오도록 하면 유지보수성이 향상됩니다.

♻️ 제안: 공통 상수로 추출

공통 인터페이스나 AbstractCsvLoader에 상수를 정의:

// AbstractCsvLoader 또는 별도의 상수 클래스에
protected static final int DATA_LOADED_THRESHOLD = 10000;

그런 다음 각 로더에서:

-	private static final String CHECK_SQL = "SELECT CASE WHEN COUNT(*) >= 10000 THEN 1 ELSE 0 END FROM p_chat_time_range";
+	private static final String CHECK_SQL = "SELECT CASE WHEN COUNT(*) >= " + DATA_LOADED_THRESHOLD + " THEN 1 ELSE 0 END FROM p_chat_time_range";

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@src/main/java/org/pgsg/user_service/user/infrastructure/loader/ChatTimeCsvLoader.java`
at line 10, The hard-coded threshold 10000 in CHECK_SQL (used in
ChatTimeCsvLoader and another loader) should be extracted to a shared constant
or made configurable: add a DATA_LOADED_THRESHOLD constant (or load from
configuration) in AbstractCsvLoader or a common constants/config class, update
ChatTimeCsvLoader.CHECK_SQL to build or parameterize the SQL using that
constant/value, and update the other loader to reference the same constant/value
so the threshold is centralized and reusable.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In
`@src/main/java/org/pgsg/user_service/user/infrastructure/loader/ChatTimeCsvLoader.java`:
- Line 10: The hard-coded threshold 10000 in CHECK_SQL (used in
ChatTimeCsvLoader and another loader) should be extracted to a shared constant
or made configurable: add a DATA_LOADED_THRESHOLD constant (or load from
configuration) in AbstractCsvLoader or a common constants/config class, update
ChatTimeCsvLoader.CHECK_SQL to build or parameterize the SQL using that
constant/value, and update the other loader to reference the same constant/value
so the threshold is centralized and reusable.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 475222ca-b74e-4932-ac7b-ba1b4c2108d4

📥 Commits

Reviewing files that changed from the base of the PR and between 0688c60 and c284390.

📒 Files selected for processing (2)

• src/main/java/org/pgsg/user_service/user/infrastructure/loader/ChatTimeCsvLoader.java
• src/main/java/org/pgsg/user_service/user/infrastructure/loader/UserCsvLoader.java