[ http://issues.apache.org/jira/browse/DERBY-1227?page=comments#action_12419566 ]
Sunitha Kambhampati commented on DERBY-1227: -------------------------------------------- I have been looking at this issue and the solution involves defining two new engine interfaces for use in the server. There is DERBY-1015 which talks about defining the engine interfaces for use in network server. I have attached a patch derby1015.diff.txt for feedback. I'd appreciate it if someone could review the changes. Thanks. > Network Server should not use the underlying embedded prepared statement when > accessing a BrokeredPreparedStatement > ------------------------------------------------------------------------------------------------------------------- > > Key: DERBY-1227 > URL: http://issues.apache.org/jira/browse/DERBY-1227 > Project: Derby > Type: Bug > Components: Network Server > Versions: 10.1.2.1 > Reporter: Kathey Marsden > Assignee: Sunitha Kambhampati > Fix For: 10.2.0.0 > > Network Server should not use the underlying embedded > Prepared statement when accessing a BrokeredPreparedStatement. > In DERBY-1025 Dan pointed out this code which while not related to DERBY-1025 > does appear to be a problem. Here is his comment regarding this code in > DRDAStatement: > /** > * Get prepared statement > * > * @return prepared statement > */ > protected PreparedStatement getPreparedStatement() throws SQLException > { > if (ps instanceof BrokeredPreparedStatement) > return (PreparedStatement)( > ((BrokeredPreparedStatement) ps).getStatement()); > else > return ps; > } > This code, for some unknown reason due to lack of comments, is getting the > underlying embedded statement > from a BrokeredPreparedStatement. This should not be allowed, the > BrokeredStatement wrappers are there to > hide the embedded statement object as it can change under the covers of the > wrapper. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira
