[
https://issues.apache.org/jira/browse/DERBY-4757?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Tiago R. Espinha updated DERBY-4757:
------------------------------------
Attachment: DERBY-4757_3.diff
I'm attaching the final version of the _3 patch. It contains mostly cosmetic
changes from the previous version and the additional fixture to the
InternationalConnectTest to test the boundaries of the new CCSID manager.
I've already ran suites.All and derbyall without any failures but I still want
to do some mixed client/server testing before committing. In the meanwhile
maybe someone could review the patch?
Knut, you're right. I just included this method definition as this was part of
its superclass, CcsidManager. I'm going to have a look at it again and if it
really isn't invoked anywhere, I'll submit a separate patch that removes this
method and the test fixture that uses it. If it's not used anywhere, it seems
pointless to have dead code laying around.
Apologies for the delays in the patches but I've been travelling lately and as
such, I've been away from regular computer access. Still, I wanted to commit
this patch before the RC for 10.7 is issued as I think it would be a nice
feature to have as part of the new version.
This would be the last patch that effectively fits everything in place for the
new CCSID manager to be used.
> (Client) Implement UTF8 support on DRDA
> ---------------------------------------
>
> Key: DERBY-4757
> URL: https://issues.apache.org/jira/browse/DERBY-4757
> Project: Derby
> Issue Type: Sub-task
> Components: Network Client
> Affects Versions: 10.7.0.0
> Reporter: Tiago R. Espinha
> Assignee: Tiago R. Espinha
> Attachments: DERBY-4757_3.diff, DERBY-4757_donotcommit.diff,
> DERBY-4757_p1.diff, DERBY-4757_p2.diff, DERBY-4757_p3.diff, DERBY-4757_p3.diff
>
>
> This issue is DERBY-4746's counterpart for the client changes required to
> implement UTF8 support in DRDA.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
