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.

Sign up for GitHub

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

Jump to bottom

[APIS-945] Fix the core dump error related with CCI Connection Pool #57

Merged

eido5 merged 7 commits into CUBRID:develop from eido5:APIS-945

Dec 6, 2022

Contributor

eido5 commented Nov 21, 2022

http://jira.cubrid.org/browse/APIS-945

Purpose

When CCI PCONNECT is used in the multi-thread environment, it must be guaranteed that application is executed whether it works failed or success. But core dump error occurs, it is required to execute the application in the multi-thread environment.

Implementation

Fix the core dump error

Remarks

N/A


          [APIS-945] Fix the core dump error related with CCI Connection Pool

85b1178

eido5 requested review from airnet73 and kisoo-han

November 21, 2022 08:07

kisoo-han reviewed

View reviewed changes

src/cci/cas_cci.c Outdated Show resolved Hide resolved

kisoo-han approved these changes

View reviewed changes

airnet73 reviewed

View reviewed changes

src/cci/cas_cci.c Outdated Show resolved Hide resolved

airnet73 reviewed

View reviewed changes

src/cci/cas_cci.c Outdated Show resolved Hide resolved


          Change the function flow

eido5 commented

View reviewed changes

src/cci/cas_cci.c Outdated Show resolved Hide resolved


          Change the function flow

a4581ad

eido5 commented

View reviewed changes

src/cci/cas_cci.c Outdated Show resolved Hide resolved

eido5 commented

View reviewed changes

src/cci/cas_cci.c Outdated Show resolved Hide resolved

eido5 requested review from kisoo-han and airnet73

November 26, 2022 03:29

airnet73 reviewed

View reviewed changes

src/cci/cas_cci.c Outdated Show resolved Hide resolved

kisoo-han reviewed

View reviewed changes

src/cci/cas_cci.c Outdated Show resolved Hide resolved

eido5 added 2 commits

December 5, 2022 20:16


          Edit the function flow and fix the core dump

64a05ea


          Rollback the previous commit

41102aa

eido5 requested a review from airnet73

December 6, 2022 02:16

airnet73 reviewed

View reviewed changes

src/cci/cas_cci.c Outdated Show resolved Hide resolved


          Edit the if statement range to filter the out of range

8b1df0a

eido5 requested a review from airnet73

December 6, 2022 07:24

eido5 commented

View reviewed changes

src/cci/cas_cci.c Outdated Show resolved Hide resolved


          Set MAX_CON_HANDLE as the public macro

airnet73 approved these changes

View reviewed changes

eido5 merged commit 87499b2 into CUBRID:develop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet