-
Notifications
You must be signed in to change notification settings - Fork 0
[FIX] 제미나이 재시도 백오프 간격 조정으로 타임아웃 개선 #98
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
Conversation
Summary of ChangesHello @lee-si-yeon, 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은 Gemini API 호출 시 발생하는 서버 타임아웃 문제를 개선하기 위해 재시도 메커니즘을 강화합니다. 재시도 횟수와 전체 제한 시간을 늘리고, 지수 백오프 전략을 조정하여 실패 시 대기 시간을 효과적으로 늘림으로써, 일시적인 네트워크 문제나 서버 부하로 인한 타임아웃에 더욱 견고하게 대응할 수 있도록 합니다. 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
안녕하세요. Gemini API 재시도 로직 변경에 대한 코드 리뷰입니다.
전반적으로 타임아웃 문제를 해결하기 위해 재시도 횟수와 백오프 간격을 조정한 점이 좋아 보입니다. 특히 지수 백오프와 jitter를 도입하여 재시도 로직을 견고하게 만든 점이 인상적입니다.
다만, 한 가지 우려되는 점이 있습니다. PR 설명과 코드 주석에는 백오프 간격이 2초, 4초, 8초로 명시되어 있지만, 실제 코드 구현은 BASE_BACKOFF_MILLIS 값으로 인해 0.4초, 0.8초, 1.6초로 동작합니다. 이는 의도한 것보다 훨씬 짧은 간격으로, 타임아웃 개선 효과가 기대에 미치지 못할 수 있습니다. 이 부분에 대한 확인이 필요해 보입니다. 자세한 내용은 코드에 남긴 개별 코멘트를 참고해주세요.
🔗 Related Issue
📝 Summary
서버 타임아웃 대응으로 Gemini 재시도 지수 백오프(2→4→8… 최대 60~90초)를 추가
🔄 Changes
💬 Questions & Review Points
📸 API Test Results (Swagger)
✅ Checklist