[ 
https://issues.apache.org/jira/browse/DERBY-4746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12891841#action_12891841
 ] 

Kathey Marsden commented on DERBY-4746:
---------------------------------------

Hi Tiago,

I looked at the patches (although in a bit of a hurry)  and they look good 
except that I am not sure about adding the DriverManager Import to ProtocolTest 
because of possible J2ME issues. It may be fine as the network server tests 
don't actually run for J2ME but I am not sure.   It might be better instead to 
use JDBCDataSource.getDataSource() and then JDBCDataSource.shutdownDatabase().

Also I don't think  public static final int EBCDIC_CCSID = 500; is used and 
might be a bit confusing since I UNICODEMGR uses 0 for EBCDIC as I recall. 
Sorry for the hurried comments.  I hope another commiter picks up the patches.

bye for now!

Kathey



> (Server) Implement UTF8 support on DRDA
> ---------------------------------------
>
>                 Key: DERBY-4746
>                 URL: https://issues.apache.org/jira/browse/DERBY-4746
>             Project: Derby
>          Issue Type: Sub-task
>    Affects Versions: 10.7.0.0
>            Reporter: Tiago R. Espinha
>            Assignee: Tiago R. Espinha
>         Attachments: DERBY-4746.diff, DERBY-4746.diff, DERBY-4746.diff, 
> DERBY-4746_p2-impl.diff, DERBY-4746_p2-tests.diff
>
>
> I'm branching out this issue into server and client changes. Since the 
> changes are incremental and small, DERBY-728 would soon become confusing if 
> it had to bear the patches for both server and client.
> For future reference, some patches have actually been applied in DERBY-728. 
> These **ARE** also required for the server implementation.

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

Reply via email to