[CBRD-26996] Fix CAS signal and cancel safety in broker#7394
Conversation
❌ TC Merge Gate — Merge BlockedOne or more TC PRs are still open. Please merge or close them before merging this PR. TC Repositories & Branches:
Steps to unblock:
|
🧪 TC Test Environment ReadyCircleCI Testing:
TC Repositories & Branches:
Next Steps:
|
|
Reviews (1): Last reviewed commit: "[CBRD-26996] Fix CAS signal and cancel s..." | Re-trigger Greptile |
| /* Process a query-cancel signal that arrived while the request header was | ||
| * being read. The actual interrupt is applied here in normal thread context, | ||
| * because signal-handler-safe functions are too limited for db_set_interrupt. */ | ||
| query_cancel_process (); |
There was a problem hiding this comment.
query_cancel_process()가 #if !defined(WINDOWS) 가드 없이 호출되고 있으나, cas_common_main.h에서의 선언은 #if !defined(WINDOWS) 블록 내부에만 있습니다. Windows 빌드에서는 함수 선언이 보이지 않아 C99 기준으로 implicit function declaration 오류가 발생합니다. cas_common_main.c에 Windows 스텁(assert(0))이 존재하더라도, 헤더에 선언이 없으면 컴파일러가 이를 거부합니다.
There was a problem hiding this comment.
큐브리드는 윈도우 빌드를 더이상 지원하지 않으므로 상관 없음.
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
|
Reviews (2): Last reviewed commit: "Merge branch 'CUBRID:develop' into featu..." | Re-trigger Greptile |
http://jira.cubrid.org/browse/CBRD-26996
Purpose
N/A
Implementation
N/A
Remarks
N/A