-
Notifications
You must be signed in to change notification settings - Fork 123
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
[CBRD-25028] Disable heap_chnguess_realloc() function, which is inaccessible from heap_Guesschn #5688
base: develop
Are you sure you want to change the base?
Conversation
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.
어떤 과정에 의해 num_clients 값이 MAX_NTRANS 값으로 초기화되는지 궁금합니다.
src/storage/heap_file.c
Outdated
static int heap_chnguess_realloc (void); | ||
#endif /* #if defined (ENABLE_UNUSED_FUNCTION) */ |
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.
Nit. #if defined
구문이 바로 위에 있기 때문에 주석을 통한 정보 제공이 불필요하다고 생각합니다.
#endif /* #if defined (ENABLE_UNUSED_FUNCTION) */ | |
#endif |
src/storage/heap_file.c
Outdated
@@ -670,7 +670,7 @@ static int heap_scancache_end_internal (THREAD_ENTRY * thread_p, HEAP_SCANCACHE | |||
#if defined (ENABLE_UNUSED_FUNCTION) | |||
static SCAN_CODE heap_get_if_diff_chn (THREAD_ENTRY * thread_p, PAGE_PTR pgptr, INT16 slotid, RECDES * recdes, | |||
bool ispeeking, int chn, MVCC_SNAPSHOT * mvcc_snapshot); | |||
#endif /* ENABLE_UNUSED_FUNCTION */ |
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.
어떤 #if 매크로와 쌍인지 쉽게 인지하기 위해서 #endif /* */ 형식을 권장합니다. 따라서, 제거하지 말아주세요. 또한 추가하신 %if ~ #endif의 #endif에도 적용해 주세요.
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.
적용했습니다.
http://jira.cubrid.org/browse/CBRD-25028
Purpose
Implementation
Remarks