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.

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-822] In JDBC Driver, translating empty string to null when the option oracleStyleEmptyString is set. #2227

Merged
merged 2 commits into from
Apr 9, 2020
Merged

[APIS-822] In JDBC Driver, translating empty string to null when the option oracleStyleEmptyString is set. #2227

merged 2 commits into from
Apr 9, 2020

Conversation

beyondykk9
Copy link
Contributor

When the option oracleStyleEmptyString is set as a connection string parameter, the empty string "" should be translated to null automatically.

This modified feature is applied to the following database types as JAVA string type:

  • CHAR
  • VARCHAR
  • CLOB

@beyondykk9 beyondykk9 added this to the damson milestone Apr 6, 2020
@beyondykk9 beyondykk9 requested a review from hgryoo April 6, 2020 00:19
@cubridci
Copy link

cubridci commented Apr 6, 2020

Build comment file:

**
** This Pull Request's title is '[APIS-822] In JDBC Driver, translating empty string to null when the option oracleStyleEmptyString is set.'.
** And it does not start with a related JIRA issue number **
** Please edit this Pull Request's title. (eg: '[CBRD-20309] Redesign vacuum data flush' **
**

hgryoo
hgryoo requested changes Apr 6, 2020
View reviewed changes
src/jdbc/cubrid/jdbc/driver/ConnectionProperties.java Outdated Show resolved Hide resolved
src/jdbc/cubrid/jdbc/jci/UStatement.java Outdated Show resolved Hide resolved
@cubridci
Copy link

cubridci commented Apr 8, 2020

Build comment file:

**
** This Pull Request's title is '[APIS-822] In JDBC Driver, translating empty string to null when the option oracleStyleEmptyString is set.'.
** And it does not start with a related JIRA issue number **
** Please edit this Pull Request's title. (eg: '[CBRD-20309] Redesign vacuum data flush' **
**

@beyondykk9 beyondykk9 self-assigned this Apr 8, 2020
@beyondykk9 beyondykk9 merged commit e9101f1 into CUBRID:develop Apr 9, 2020
@beyondykk9 beyondykk9 mentioned this pull request Apr 16, 2020
Merged
beyondykk9 added a commit that referenced this pull request Apr 16, 2020
@beyondykk9
[APIS-822] In JDBC Driver, translating empty string to null when the …
658b18c 
…option oracleStyleEmptyString is set. (#2227) (#2249)

* [APIS-822] In JDBC Driver, translating empty string to null when the option oracleStyleEmptyString is set.

* [APIS-822] revised to remove unnecessary code
@beyondykk9 beyondykk9 mentioned this pull request Apr 21, 2020
Merged
beyondykk9 added a commit that referenced this pull request Apr 21, 2020
@beyondykk9
[APIS-822] In JDBC Driver, translating empty string to null when the …
48e4744 
…option oracleStyleEmptyString is set. (#2227) (#2258)

* [APIS-822] In JDBC Driver, translating empty string to null when the option oracleStyleEmptyString is set.

* [APIS-822] revised to remove unnecessary code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hgryoo hgryoo hgryoo approved these changes

@shparkcubrid shparkcubrid shparkcubrid approved these changes

Assignees

@beyondykk9 beyondykk9

Labels
enhancement
Projects
None yet
Milestone
damson
Development

Successfully merging this pull request may close these issues.
4 participants
@beyondykk9 @cubridci @hgryoo @shparkcubrid