소프트웨어 아키텍처 the hard parts sprint 3 - 김종필#601
Conversation
|
우측에 있는 |
Summary of ChangesHello @jongfeel, 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! 이 PR은 "소프트웨어 아키텍처 the hard parts 2판"의 6장과 7장에 대한 저자의 요약 및 리뷰 노트를 추가합니다. 이 노트들은 데이터베이스 특성, 도메인 주도 설계의 바운디드 컨텍스트의 미묘한 차이, 그리고 마이크로서비스 아키텍처에서 서비스 세분도를 결정할 때 객관적인 기준의 중요성을 깊이 있게 다룹니다. 저자는 또한 각 장에서 얻은 개인적인 통찰력과 논의 주제를 공유합니다. 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. Changelog
Activity
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.
Code Review
안녕하세요, 김종필님. PR 잘 보았습니다. '소프트웨어 아키텍처 the hard parts' 6장과 7장에 대한 리뷰를 추가하셨네요. 책의 내용을 깊이 있게 이해하고 자신의 경험과 연결하여 정리한 점이 인상적입니다. 특히 서비스 분해 및 통합에 대한 객관적인 기준의 필요성을 강조하고, 과거 경험을 공유해주신 부분이 다른 분들에게도 큰 도움이 될 것 같습니다. 문서의 완성도를 높이기 위해 몇 가지 개선점을 제안드립니다.
|
|
||
| 아마 오라클 데이터베이스를 쓸 때 알았을 수도 있지만, 현 시점에서는 처음 들어보는 용어이다. | ||
| AI에게 물어보거나 검색해 보면 어떤 용도로 쓰는지 다시 복습할 수 있다. | ||
| 핵심 개념은 다른 데이터베이스를 참조하거나 다른 서버의 데이터베이스를 참조해서 쓰고 싶을 때 정의해서 사용하는 것이다. No newline at end of file |
There was a problem hiding this comment.
| - https://github.com/jongfeel/BookReview/issues/1612#issuecomment-3831102227 | ||
| - https://github.com/jongfeel/BookReview/issues/1612#issuecomment-3841314191 |
| 서비스 세분도의 분해인, 통합인의 쉽고 이해할 수 있는 설명에 감동받아 | ||
| 과거 서비스의 세분도를 떠올려 봤을 때 이 책에서 설명한 분해인, 통합인에 해당됐던 것들이 있는지 얘기해 보면 좋을 것 같습니다. | ||
|
|
||
| 저의 경우는 인프라 관련 횡단 기능인 API 공통 로직에 비즈니스적인 요구사항의 기능을 조금씩 넣었고 그걸 계속 하나의 서비스로 유지했다가 나중에 분리하려고 하면서 이 책에서 설명하는 경계 콘텍스트를 나누지 못해 결국 분리를 하지 못한 경험이 있습니다. No newline at end of file |
There was a problem hiding this comment.
파일 끝에 개행 문자가 없습니다. 일반적으로 텍스트 파일은 끝에 개행 문자를 추가하는 것이 좋은 관행입니다. 이는 일부 도구와의 호환성을 보장하고 파일 끝을 명확하게 표시하는 데 도움이 됩니다.
| 저의 경우는 인프라 관련 횡단 기능인 API 공통 로직에 비즈니스적인 요구사항의 기능을 조금씩 넣었고 그걸 계속 하나의 서비스로 유지했다가 나중에 분리하려고 하면서 이 책에서 설명하는 경계 콘텍스트를 나누지 못해 결국 분리를 하지 못한 경험이 있습니다. | |
| 저의 경우는 인프라 관련 횡단 기능인 API 공통 로직에 비즈니스적인 요구사항의 기능을 조금씩 넣었고 그걸 계속 하나의 서비스로 유지했다가 나중에 분리하려고 하면서 이 책에서 설명하는 경계 콘텍스트를 나누지 못해 결국 분리를 하지 못한 경험이 있습니다. | |
NOTICE
다들 pull request의 제목을 주차로 적었는데 저희 모임은 2주 간격으로 진행됩니다.
따라서 산술적으로는 이번이 6주차가 맞는데 주차로 적으실 분은 그렇게 해도 되고 저는 sprint라는 단위를 쓰는게 좋다고 봐서 sprint로 바꿔서 쓰겠습니다.
Review
이번에도 여러 데이터베이스의 특성에 대해 알 수 있는 시간도 가져보고
마이크로 서비스라고 서비스를 무작정 나누지 말고 어떤 요인으로 나누고 통합해야 하는지 분석하는 방법을 또 친절하고 쉽게 알려줘서 이번에도 큰 감동을 받았습니다.
지금 새벽시간인데도 책을 끝까지 읽고 읽었던 내용을 다시 생각하게 만드는 힘이 있는 좋은 책인 것 같습니다.