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

[김대겸] Step3 PR #601

Merged
merged 9 commits into from Dec 11, 2022
Merged

[김대겸] Step3 PR #601

merged 9 commits into from Dec 11, 2022

Conversation

Gyeom
Copy link

@Gyeom Gyeom commented Dec 10, 2022

안녕하세요 리뷰어님~

3단계 - 로깅, 모니터링 리뷰 요청 드립니다.

이번에도 리뷰 잘 부탁드립니다 🙏 🙏 🙏

Copy link

@geunhwanlee geunhwanlee left a comment

Choose a reason for hiding this comment

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

3단계 미션 잘 해주셨습니다. 👍🏻

충분히 잘해주셔서 머지하도록 할게요

미션 진행하느라 고생 많으셨습니다.
남은 미션들도 잘 마무리하시길 바래요 😀

Comment on lines +11 to +16
<rollingPolicy class="ch.qos.logback.core.rolling.TimeBasedRollingPolicy">
<fileNamePattern>${home}json-%d{yyyyMMdd}-%i.log</fileNamePattern>
<timeBasedFileNamingAndTriggeringPolicy class="ch.qos.logback.core.rolling.SizeAndTimeBasedFNATP">
<maxFileSize>15MB</maxFileSize>
</timeBasedFileNamingAndTriggeringPolicy>
</rollingPolicy>

Choose a reason for hiding this comment

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

날짜가 넘어가거나 사이즈가 15MB를 넘어가면 새로운 파일로 넘어가게 설정하셨네요 👍🏻
하지만 오랫동안 서버를 유지하게된다면 로그파일이 무한정 만들어지게 될텐데요
그럴경우 어떻게 처리를 하는게 좋을지 고민해보면 좋겠네요 😀

@geunhwanlee geunhwanlee merged commit a003a30 into next-step:gyeom Dec 11, 2022
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.

None yet

2 participants