Develop

[jaebeom79]

📢 기능 설명

필요시 실행결과 스크린샷 첨부

연결된 issue

연결된 issue를 자동을 닫기 위해 아래 {이슈넘버}를 입력해주세요.

close #{이슈넘버}

✅ 체크리스트

• PR 제목 규칙 잘 지켰는가?
• 추가/수정사항을 설명하였는가?
• 이슈넘버를 적었는가?

[gemini-code-assist]

Code Review

This pull request introduces a comprehensive monitoring stack including Prometheus, Loki, Promtail, Jaeger, and Grafana, alongside OpenTelemetry integration for automated tracing. It also optimizes the StoreRemain slot generation logic by implementing pre-calculated slot plans and using interface projections to improve database query efficiency. Security improvements include restricting external access to Actuator endpoints via Nginx. Review feedback suggests evaluating alternative Garbage Collectors (Serial or Parallel) for the small heap size, mitigating security risks by avoiding the root user for the Jaeger container, and implementing null checks before parsing store operating hours to prevent potential exceptions.

[gemini-code-assist]

While -XX:+UseG1GC is the default in Java 21, explicitly setting it for a container limited to 600MB (as seen in docker-compose.prod.yml) might not be optimal. G1GC is generally more effective for larger heaps. For a heap size around 400MB (70% of 600MB), you might consider testing SerialGC or ParallelGC to see if they provide better performance with lower overhead in this resource-constrained environment.

[gemini-code-assist]

Running the Jaeger container as root (user: "0") is a security risk, even for internal tools. It is better to ensure the host directory has the correct permissions for the container's default user or to specify a non-root UID that matches the host's directory owner.

[gemini-code-assist]

The LocalTime.parse method will throw a NullPointerException if store.getOpenTime() or store.getCloseTime() returns null. Although there is a catch block, it is safer and more idiomatic to perform a null check before parsing to avoid unnecessary exception overhead.

                String openTimeStr = store.getOpenTime();
                String closeTimeStr = store.getCloseTime();
                if (openTimeStr == null || closeTimeStr == null) {
                    log.warn("[영업시간 누락] storeId={}", store.getId());
                    continue;
                }
                LocalTime openTime = LocalTime.parse(openTimeStr, TIME_FORMATTER);
                LocalTime closeTime = LocalTime.parse(closeTimeStr, TIME_FORMATTER);