-
Notifications
You must be signed in to change notification settings - Fork 5
Tttghost/software architecture the hard parts chapter1 3 #580
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
Tttghost/software architecture the hard parts chapter1 3 #580
Conversation
Removed a line requesting a final review.
|
우측에 있는 |
Summary of ChangesHello @tttghost, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 Pull Request는 '소프트웨어 아키텍처 The Hard Parts' 책의 1장부터 3장까지의 내용을 요약하고, 저자의 실제 개발 경험을 바탕으로 다양한 아키텍처 스타일과 원칙에 대한 깊이 있는 통찰을 제공합니다. 아키텍처 결정의 핵심인 트레이드오프 개념을 강조하며, 실무에서 마주하는 아키텍처 문제 해결에 대한 이해를 돕습니다. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
소프트웨어 아키텍처 책의 1-3장 내용 정리를 위한 PR이군요. 책 내용이 어렵다고 하셨는데, 꼼꼼하게 정리하신 내용 잘 읽었습니다. 두 분이 작성하신 마크다운 파일에서 가독성을 조금 더 향상시킬 수 있는 부분들에 대해 몇 가지 의견을 남겼습니다. 주로 문장 중간의 줄바꿈이나 불필요한 공백을 수정하여 글의 흐름을 더 자연스럽게 만드는 것에 대한 제안입니다. 확인 부탁드립니다.
| - 사내에 FDC(financial Data Center)가 존재하고, 매일 bulk로 사내 전체 DB를 조회하여 적재한다. 그리고 매일 배치를 돌려서 누락된 적립이 있는지 확인하고, 존재한다면 노티 이메일을 보내준다. 따라서 | ||
| 재처리가 가능하다. 이것이 운영에서 가장 중요한 포인트였다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
문장의 흐름이 끊어지지 않도록 한 문장으로 합치는 것이 좋겠습니다. '따라서' 뒤의 줄바꿈을 제거하여 가독성을 높일 수 있습니다.
| - 사내에 FDC(financial Data Center)가 존재하고, 매일 bulk로 사내 전체 DB를 조회하여 적재한다. 그리고 매일 배치를 돌려서 누락된 적립이 있는지 확인하고, 존재한다면 노티 이메일을 보내준다. 따라서 | |
| 재처리가 가능하다. 이것이 운영에서 가장 중요한 포인트였다. | |
| - 사내에 FDC(financial Data Center)가 존재하고, 매일 bulk로 사내 전체 DB를 조회하여 적재한다. 그리고 매일 배치를 돌려서 누락된 적립이 있는지 확인하고, 존재한다면 노티 이메일을 보내준다. 따라서 재처리가 가능하다. 이것이 운영에서 가장 중요한 포인트였다. |
| 데이터는 모든 것: 데이터 설계가 가장 중요하다. | ||
| 운영데이터와 분석데이터가 있다. | ||
| 운영데이터는 회사 시스템이 돌아가는 데 필요한 데이터다. 정합성, 트랜잭션이 중요 | ||
| 분석데이터는 서비스를 이해, 개선, 판단을 위한 데이터로 트랜잭션과 무관하며 과거 누적 축적형 데이터로 현재 운영에 필요하진 않지만 장기적인 전략 수립 및 의사결정에 중요하게 활용되는 데이터다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| 피트니스 함수에서 자동화한 부분(소나큐브, 아크유닛 등)과 수동으로 검토해야 하는 부분이 | ||
| 있다고 나오는데 실무에서 이 두 부분에 대한 경험이 있으신가요? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
안녕하세요. main이 아니라 저의 branch(geunju-lee/20260107)에서 새 branch를 따서 저의 md 파일까지 pr에 올라간거 같습니다. |
브랜치잘못땄네요 -ㅁ-;; 죄송합니다! |
책 내용이 정말 어렵네요...
용어 숙지해가며 읽느라 시간이 더 오래걸리는 듯 합니다.