[DEV-385/BE] feat: web client 응답 시간 제한 3분으로 증가#620
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
kckc0608
requested review from
lja3723 and
zxc534
and removed request for
zxc534
Summary of ChangesHello @kckc0608, 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은 질답 변환 프로세스에 필요한 시간을 확보하기 위해 시스템의 타임아웃 설정을 조정합니다. 특히, 웹 클라이언트의 외부 서비스 요청 타임아웃을 3분으로 늘려 안정적인 응답을 보장하고, 내부 질답 세트 변환 결과 타임아웃도 2분으로 확장하여 프로세스 완료에 충분한 시간을 제공합니다. Highlights
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
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
이 PR은 질답 변환에 더 긴 처리 시간을 허용하기 위해 웹 클라이언트의 응답 시간 제한을 3분으로 늘리는 것을 목표로 합니다. 변경 사항에는 application-gemini.yml의 설정 값과 관련 상수를 업데이트하는 내용이 포함되어 있습니다. 웹 클라이언트의 시간 제한은 180초로 올바르게 설정되었지만, 관련된 다른 시간 제한 상수가 2분으로 설정된 불일치를 발견했습니다. 이는 클라이언트에서 조기 타임아웃을 유발할 수 있습니다. 이 불일치를 해결하기 위한 리뷰 의견을 남겼습니다.
| public static final int RAW_TEXT_MAX_LENGTH = 10_000; | ||
| public static final int KPT_TEXT_MAX_LENGTH = 8_000; | ||
| public static final long QNA_SET_CONVERT_RESULT_TIMEOUT_MILLISECONDS = 30_000L; | ||
| public static final long QNA_SET_CONVERT_RESULT_TIMEOUT_MILLISECONDS = 120_000L; |
There was a problem hiding this comment.
PR 제목과 application-gemini.yml 변경 내용에 따르면 타임아웃을 3분(180초)으로 설정하려는 것으로 보입니다. 하지만 이 상수는 120,000ms (2분)으로 설정되어 있습니다. WebClient 요청의 타임아웃은 3분인데, 이 상수가 사용될 가능성이 있는 DeferredResult의 타임아웃이 2분이므로 클라이언트 요청이 백엔드 작업이 완료되기도 전에 타임아웃될 수 있습니다. 일관성을 유지하고 조기 타임아웃을 방지하기 위해 이 값도 3분으로 설정하는 것이 좋겠습니다.
| public static final long QNA_SET_CONVERT_RESULT_TIMEOUT_MILLISECONDS = 120_000L; | |
| public static final long QNA_SET_CONVERT_RESULT_TIMEOUT_MILLISECONDS = 180_000L; |
forhyundaisofteer
deleted the
DEV-385/feat/web-client-응답-시간-제한-3분으로-해제
branch
1 participant
관련 이슈
close #619
작업한 내용
PR 리뷰시 참고할 사항
참고 자료 (링크, 사진, 예시 코드 등)