Skip to content
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-24499] Setting CUBRID_TMP causes nativesocket bind error in Java a SP (#3871) #3878

Merged
merged 1 commit into from
Oct 17, 2022

Conversation

hgryoo
Copy link
Member

@hgryoo hgryoo commented Oct 14, 2022

http://jira.cubrid.org/browse/CBRD-24499

Note
This patch was reverted because of CI hang problem. But the problem is not from the commit.

Purpose
Setting CUBRID_TMP causes a native socket bind error because the UDS socket file is not created properly because the path is set wrongly.

Implementation
I've changed the UDS socket file path for javasp same as the broker's socket file inside of $CUBRID/var/CUBRID_SOCK/ if CUBRID_TMP is not set. If the CUBRID_TMP is set, the socket file will be created inside of the $CUBRID_TMP directory. The exact path of the UDS socket file is created in the cub_javasp utility, and it passes to the JVM.

…a SP (CUBRID#3871)

http://jira.cubrid.org/browse/CBRD-24499

*Purpose*
Setting CUBRID_TMP causes a native socket bind error because the UDS socket file is not created properly because the path is set wrongly.

*Implementation*
I've changed the UDS socket file path for javasp same as the broker's socket file inside of $CUBRID/var/CUBRID_SOCK/ if CUBRID_TMP is not set. If the CUBRID_TMP is set, the socket file will be created inside of the $CUBRID_TMP directory.
The exact path of the UDS socket file is created in the cub_javasp utility, and it passes to the JVM.
@hgryoo hgryoo requested a review from beyondykk9 October 14, 2022 12:14
@hgryoo hgryoo self-assigned this Oct 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants