[
https://issues.apache.org/jira/browse/DERBY-6801?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Bryan Pendleton updated DERBY-6801:
-----------------------------------
Attachment: unsealDerbyNetJar.diff
My test failures involved running with jars, and it turns out
that we also need to unseal o.a.d.shared.common.error
in derbynet.jar, because the latest patches introduce use
of that package from derbynet.jar classes.
Since o.a.d.shared.common.error classes are designed
to be shared freely among both client and server code,
this is as expected, so the attached unsealDerbyNetJar.diff
unseals the package in that jar.
And now the derbynet tests are running better on my system.
Abhinav, have you had an opportunity to investigate the
revised MessageUtils patch I attached yesterday?
> Implement MessageUtils class so client and server can share message argument
> encoding/decoding
> ----------------------------------------------------------------------------------------------
>
> Key: DERBY-6801
> URL: https://issues.apache.org/jira/browse/DERBY-6801
> Project: Derby
> Issue Type: Sub-task
> Components: JDBC, Network Client, Network Server
> Reporter: Bryan Pendleton
> Assignee: Abhinav Gupta
> Priority: Minor
> Attachments: MessageUtilsBryan.diff, MessageUtils_2.diff,
> MessageUtils_3.diff, MessageUtils_StandardEx_1.diff,
> MessageUtils_StandardExcep.diff, build.out, messageUtils.diff,
> messageUtils_1.diff, subPart2.diff, unsealDerbyNetJar.diff
>
>
> Currently, the Network Server contains code which encodes message arguments
> to be passed over DRDA to the client, and other code which
> decodes the encoded message arguments when the client passes
> them back in a call to SQLCAMESSAGE (see DERBY-1178 for additional
> discussion of this behavior).
> We would like to extract that logic and place it into a new MessageUtils
> class in the common shared library so that both the client and server
> can manipulate the message arguments without duplicating the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
