db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kristian Waagan (JIRA)" <j...@apache.org>
Subject [jira] Closed: (DERBY-2818) Rewrite ClobUpdatableReader constructors
Date Fri, 15 Jun 2007 13:06:26 GMT

     [ https://issues.apache.org/jira/browse/DERBY-2818?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel

Kristian Waagan closed DERBY-2818.

No problems reported by the nightlies. Closing the issue.

> Rewrite ClobUpdatableReader constructors
> ----------------------------------------
>                 Key: DERBY-2818
>                 URL: https://issues.apache.org/jira/browse/DERBY-2818
>             Project: Derby
>          Issue Type: Improvement
>          Components: JDBC
>    Affects Versions:
>            Reporter: Kristian Waagan
>            Assignee: Kristian Waagan
>             Fix For:
>         Attachments: derby-2818-1a.diff, derby-2818-1a.stat
> When the fix for DERBY-2806 is in place, the current implementation of ClobUpdateableReader
will fail.
> This patch is a preparation for the mentioned issue, and I have also removed some "internal"
(package private) methods from EmbedClob and replaced it with a single one. 
> Instead of writing methods that forwards calls to the underlying InternalClob, the method
getInternalClob will return the internal clob itself. This seems more in place, since the
method getByteLength does not belong in EmbedClob. It also comes with a warning that using
the reference to the internal clob object requires the caller to take more care. Second, isWritable()
does not belong to EmbedClob either, as a Clob is always writable (unless the database/connection
itself is read/only). However, the internal clob representation may be read-only, in case
EmbedClob must clone the content and create a writable representation.
> The fix leaves more of the handling of implementation details to the InternalClob itself.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message