Skip to content

Bugfix/dbms lob append multibyte #348

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
Merged
merged 3 commits into from
Jun 12, 2017
Merged

Bugfix/dbms lob append multibyte #348

merged 3 commits into from
Jun 12, 2017

Conversation

jgebal
Copy link
Member

@jgebal jgebal commented Jun 9, 2017

Fixed #346
Fixed issue with length semantic in ut_utils.append_to_clob - replaced length with dbms_lob.getlength.
Replaced direct calls to dbms_lob.writeappend with calls to ut_utils.append_to_clob

jgebal added 2 commits June 9, 2017 17:24
@jgebal
Provided a test to reproduce the issue with length function not wor…
78a0d41 
…king properly.
@jgebal
Fixed #346
9b755d5 
Fixed issue with length semantic in `ut_utils.append_to_clob` - replaced length with dbms_lob.getlength.
Replaced direct calls to `dbms_lob.writeappend` with calls to `ut_utils.append_to_clob`
@jgebal jgebal added the bugfix label Jun 9, 2017
@jgebal jgebal added this to the v3.0.1 milestone Jun 9, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 91.708% when pulling 9b755d5 on jgebal:bugfix/dbms_lob_append_multibyte into d160610 on utPLSQL:develop.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage remained the same at 91.708% when pulling 9b755d5 on jgebal:bugfix/dbms_lob_append_multibyte into d160610 on utPLSQL:develop.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 91.708% when pulling 9b755d5 on jgebal:bugfix/dbms_lob_append_multibyte into d160610 on utPLSQL:develop.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 91.708% when pulling bcb928e on jgebal:bugfix/dbms_lob_append_multibyte into cb4fa1b on utPLSQL:develop.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage remained the same at 91.708% when pulling bcb928e on jgebal:bugfix/dbms_lob_append_multibyte into cb4fa1b on utPLSQL:develop.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 91.708% when pulling bcb928e on jgebal:bugfix/dbms_lob_append_multibyte into cb4fa1b on utPLSQL:develop.

@Pazus Pazus merged commit a43e3ff into utPLSQL:develop Jun 12, 2017
@Pazus Pazus removed the in progress label Jun 12, 2017
@jgebal jgebal deleted the bugfix/dbms_lob_append_multibyte branch June 12, 2017 19:52
@pocelka pocelka mentioned this pull request Jun 14, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Pazus Pazus Pazus approved these changes

Assignees
No one assigned
Labels
bugfix
Projects
None yet
Milestone
v3.0.1
Development

Successfully merging this pull request may close these issues.
3 participants
@jgebal @coveralls @Pazus