You are viewing a plain text version of this content. The canonical link for it is here.
Posted to derby-dev@db.apache.org by "Kristian Waagan (JIRA)" <ji...@apache.org> on 2008/02/21 21:17:20 UTC
[jira] Created: (DERBY-3446) Make ResultSet.getStatemet return the
correct statement when created by a logical statement
Make ResultSet.getStatemet return the correct statement when created by a logical statement
-------------------------------------------------------------------------------------------
Key: DERBY-3446
URL: https://issues.apache.org/jira/browse/DERBY-3446
Project: Derby
Issue Type: Task
Components: JDBC, Network Client
Affects Versions: 10.4.0.0
Reporter: Kristian Waagan
Assignee: Kristian Waagan
Priority: Minor
Fix For: 10.4.0.0
ResultSet.getStatement must return the correct statement, that is the statement that created the result set.
It is particularly important for result set created by logical statements, as leaking of physical statements can cause all kinds of side effects in a connection pooling environment.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (DERBY-3446) Make ResultSet.getStatemet return the
correct statement when created by a logical statement
Posted by "Kristian Waagan (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-3446?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kristian Waagan updated DERBY-3446:
-----------------------------------
Derby Info: [Patch Available]
> Make ResultSet.getStatemet return the correct statement when created by a logical statement
> -------------------------------------------------------------------------------------------
>
> Key: DERBY-3446
> URL: https://issues.apache.org/jira/browse/DERBY-3446
> Project: Derby
> Issue Type: Task
> Components: JDBC, Network Client
> Affects Versions: 10.4.1.3
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.4.1.3
>
> Attachments: derby-3446-1a_rs_getstatement.diff, derby-3446-1a_rs_getstatement.stat, derby-3446-2a_rs_getstatement_alternative.diff, derby-3446-2a_rs_getstatement_alternative.stat, derby-3446-2b_rs_getstatement_alternative.diff, derby-3446-2c_rs_getstatement_alternative.diff, derby-3446-3a-stmtpool_test.diff
>
>
> ResultSet.getStatement must return the correct statement, that is the statement that created the result set.
> It is particularly important for result set created by logical statements, as leaking of physical statements can cause all kinds of side effects in a connection pooling environment.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (DERBY-3446) Make ResultSet.getStatemet return
the correct statement when created by a logical statement
Posted by "Knut Anders Hatlen (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-3446?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12571365#action_12571365 ]
Knut Anders Hatlen commented on DERBY-3446:
-------------------------------------------
Patch 2a is exactly what I was thinking, except I assumed LogicalStatementEntity that was implementing Statement. Since all sub-classes of LogicalStatementEntity are implementing Statement, and LogicalStatementEntity is an abstract class (actually, it's not declared as abstract, but in reality it is used as an abstract class I think), either of these approaches should work and make the code even simpler:
a) Declare LogicalStatementEntity as an abstract class than implements java.sql.Statement, and move calls to setOwner() from the sub-classes' constructors into LSE's constructor.
b) Remove calls to setOwner() from the sub-classes, and do this in LSE's constructor: physicalPS.setOwner((java.sql.Statement) this);
I think I prefer (a) since these two approaches are essentially identical, only (a) makes the assumptions verifiable at compile time.
> Make ResultSet.getStatemet return the correct statement when created by a logical statement
> -------------------------------------------------------------------------------------------
>
> Key: DERBY-3446
> URL: https://issues.apache.org/jira/browse/DERBY-3446
> Project: Derby
> Issue Type: Task
> Components: JDBC, Network Client
> Affects Versions: 10.4.0.0
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.4.0.0
>
> Attachments: derby-3446-1a_rs_getstatement.diff, derby-3446-1a_rs_getstatement.stat, derby-3446-2a_rs_getstatement_alternative.diff, derby-3446-2a_rs_getstatement_alternative.stat
>
>
> ResultSet.getStatement must return the correct statement, that is the statement that created the result set.
> It is particularly important for result set created by logical statements, as leaking of physical statements can cause all kinds of side effects in a connection pooling environment.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (DERBY-3446) Make ResultSet.getStatemet return the
correct statement when created by a logical statement
Posted by "Kristian Waagan (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-3446?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kristian Waagan updated DERBY-3446:
-----------------------------------
Attachment: derby-3446-2b_rs_getstatement_alternative.diff
Attached 'derby-3446-2b_rs_getstatement_alternative.diff'.
a) Actually, I had LogicalStatementEntity as an abstract class, until a tool told me it shouldn't be because it didn't have any abstract methods. So I removed the abstract keyword and made the constructor protected instead... I didn't let LSE implement java.sql.Statement, because I saw no need for it (until now).
It does make sense with the current plan, and I have implemented approach a in patch 2b.
Thanks for the input :)
I'm also wondering if it would make sense to refer to the prepared statements as am.PreparedStatement, since casts are starting to pop up in the code. Is the code a little too generic? This would be handled in another issue though, and it needs to investigated.
> Make ResultSet.getStatemet return the correct statement when created by a logical statement
> -------------------------------------------------------------------------------------------
>
> Key: DERBY-3446
> URL: https://issues.apache.org/jira/browse/DERBY-3446
> Project: Derby
> Issue Type: Task
> Components: JDBC, Network Client
> Affects Versions: 10.4.0.0
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.4.0.0
>
> Attachments: derby-3446-1a_rs_getstatement.diff, derby-3446-1a_rs_getstatement.stat, derby-3446-2a_rs_getstatement_alternative.diff, derby-3446-2a_rs_getstatement_alternative.stat, derby-3446-2b_rs_getstatement_alternative.diff
>
>
> ResultSet.getStatement must return the correct statement, that is the statement that created the result set.
> It is particularly important for result set created by logical statements, as leaking of physical statements can cause all kinds of side effects in a connection pooling environment.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (DERBY-3446) Make ResultSet.getStatemet return
the correct statement when created by a logical statement
Posted by "Knut Anders Hatlen (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-3446?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12571318#action_12571318 ]
Knut Anders Hatlen commented on DERBY-3446:
-------------------------------------------
The patch looks fine to me. Two small questions:
Does ResultSet.creatingStatement have to be volatile? It seems like it's always set before the ResultSet is returned to the user, and not modified later, so I can't see that there are any threading/visibility issues even if it's not volatile.
Have you considered this alternative approach:
- have a field called owner in am.Statement
- in LogicalStatementEntity's constructor, cast physicalPS to am.PreparedStatement and set the owner
- implement am.ResultSet.getStatement() as "if (statement_.owner != null) return statement_.owner; else return statement_;"
I may have overlooked something, but it sounds like this requires even fewer changes, and, more importantly, it's more robust as it will work automatically if more methods that return ResultSets are added to the JDBC interfaces later.
> Make ResultSet.getStatemet return the correct statement when created by a logical statement
> -------------------------------------------------------------------------------------------
>
> Key: DERBY-3446
> URL: https://issues.apache.org/jira/browse/DERBY-3446
> Project: Derby
> Issue Type: Task
> Components: JDBC, Network Client
> Affects Versions: 10.4.0.0
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.4.0.0
>
> Attachments: derby-3446-1a_rs_getstatement.diff, derby-3446-1a_rs_getstatement.stat
>
>
> ResultSet.getStatement must return the correct statement, that is the statement that created the result set.
> It is particularly important for result set created by logical statements, as leaking of physical statements can cause all kinds of side effects in a connection pooling environment.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (DERBY-3446) Make ResultSet.getStatement return the
correct statement when created by a logical statement
Posted by "Kristian Waagan (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-3446?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kristian Waagan updated DERBY-3446:
-----------------------------------
Derby Info: (was: [Patch Available])
Fix Version/s: (was: 10.4.1.3)
10.5.0.0
10.4.1.4
Summary: Make ResultSet.getStatement return the correct statement when created by a logical statement (was: Make ResultSet.getStatemet return the correct statement when created by a logical statement)
Committed patch 3a to trunk with revision 673327.
Backported to 10.4 with revision 673329.
Updated fix versions and fixed typo in the summary.
The addition of the test concludes the work on this issue.
> Make ResultSet.getStatement return the correct statement when created by a logical statement
> --------------------------------------------------------------------------------------------
>
> Key: DERBY-3446
> URL: https://issues.apache.org/jira/browse/DERBY-3446
> Project: Derby
> Issue Type: Task
> Components: JDBC, Network Client
> Affects Versions: 10.4.1.3
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.4.1.4, 10.5.0.0
>
> Attachments: derby-3446-1a_rs_getstatement.diff, derby-3446-1a_rs_getstatement.stat, derby-3446-2a_rs_getstatement_alternative.diff, derby-3446-2a_rs_getstatement_alternative.stat, derby-3446-2b_rs_getstatement_alternative.diff, derby-3446-2c_rs_getstatement_alternative.diff, derby-3446-3a-stmtpool_test.diff
>
>
> ResultSet.getStatement must return the correct statement, that is the statement that created the result set.
> It is particularly important for result set created by logical statements, as leaking of physical statements can cause all kinds of side effects in a connection pooling environment.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (DERBY-3446) Make ResultSet.getStatemet return the
correct statement when created by a logical statement
Posted by "Kristian Waagan (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-3446?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kristian Waagan updated DERBY-3446:
-----------------------------------
Derby Info: (was: [Patch Available])
> Make ResultSet.getStatemet return the correct statement when created by a logical statement
> -------------------------------------------------------------------------------------------
>
> Key: DERBY-3446
> URL: https://issues.apache.org/jira/browse/DERBY-3446
> Project: Derby
> Issue Type: Task
> Components: JDBC, Network Client
> Affects Versions: 10.4.0.0
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.4.0.0
>
> Attachments: derby-3446-1a_rs_getstatement.diff, derby-3446-1a_rs_getstatement.stat, derby-3446-2a_rs_getstatement_alternative.diff, derby-3446-2a_rs_getstatement_alternative.stat, derby-3446-2b_rs_getstatement_alternative.diff, derby-3446-2c_rs_getstatement_alternative.diff
>
>
> ResultSet.getStatement must return the correct statement, that is the statement that created the result set.
> It is particularly important for result set created by logical statements, as leaking of physical statements can cause all kinds of side effects in a connection pooling environment.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (DERBY-3446) Make ResultSet.getStatemet return
the correct statement when created by a logical statement
Posted by "Knut Anders Hatlen (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-3446?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12571374#action_12571374 ]
Knut Anders Hatlen commented on DERBY-3446:
-------------------------------------------
Regarding gc, we don't do any clean-up in the logical statement's finalizer, and the only resource the logical statement holds on to is the physical statement. This means that the reference from the physical statement which could keep the logical statement alive a while longer, only affects the lifetime of the physical statement. So it's kind of a self reference, and once all other references to the physical statement are gone, it can be gc'ed (except when the logical statement is still open and referenced, but that's not changed with this patch). If it turns out to be a problem, it can probably be fixed easily with a weak reference.
If the user remembers to close the logical statement there will definitely not be any problem at all, since you reset the reference in LSE.close().
> Make ResultSet.getStatemet return the correct statement when created by a logical statement
> -------------------------------------------------------------------------------------------
>
> Key: DERBY-3446
> URL: https://issues.apache.org/jira/browse/DERBY-3446
> Project: Derby
> Issue Type: Task
> Components: JDBC, Network Client
> Affects Versions: 10.4.0.0
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.4.0.0
>
> Attachments: derby-3446-1a_rs_getstatement.diff, derby-3446-1a_rs_getstatement.stat, derby-3446-2a_rs_getstatement_alternative.diff, derby-3446-2a_rs_getstatement_alternative.stat, derby-3446-2b_rs_getstatement_alternative.diff
>
>
> ResultSet.getStatement must return the correct statement, that is the statement that created the result set.
> It is particularly important for result set created by logical statements, as leaking of physical statements can cause all kinds of side effects in a connection pooling environment.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (DERBY-3446) Make ResultSet.getStatemet return the
correct statement when created by a logical statement
Posted by "Kristian Waagan (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-3446?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kristian Waagan updated DERBY-3446:
-----------------------------------
Attachment: derby-3446-2c_rs_getstatement_alternative.diff
Attached 'derby-3446-2c_rs_getstatement_alternative.diff', which is the same as 2b except the tests have been removed. I don't think the tests are in the correct location, as they will only be executed for the embedded driver with the current patch. I will a also look into enabling the tests for both CPDS and XADS.
I will see if there is a better place for the tests.
Committed patch 2c to trunk with revision 630784.
> Make ResultSet.getStatemet return the correct statement when created by a logical statement
> -------------------------------------------------------------------------------------------
>
> Key: DERBY-3446
> URL: https://issues.apache.org/jira/browse/DERBY-3446
> Project: Derby
> Issue Type: Task
> Components: JDBC, Network Client
> Affects Versions: 10.4.0.0
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.4.0.0
>
> Attachments: derby-3446-1a_rs_getstatement.diff, derby-3446-1a_rs_getstatement.stat, derby-3446-2a_rs_getstatement_alternative.diff, derby-3446-2a_rs_getstatement_alternative.stat, derby-3446-2b_rs_getstatement_alternative.diff, derby-3446-2c_rs_getstatement_alternative.diff
>
>
> ResultSet.getStatement must return the correct statement, that is the statement that created the result set.
> It is particularly important for result set created by logical statements, as leaking of physical statements can cause all kinds of side effects in a connection pooling environment.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Closed: (DERBY-3446) Make ResultSet.getStatement return the
correct statement when created by a logical statement
Posted by "Kristian Waagan (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-3446?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kristian Waagan closed DERBY-3446.
----------------------------------
Fix implemented, test runs cleanly.
Closing.
> Make ResultSet.getStatement return the correct statement when created by a logical statement
> --------------------------------------------------------------------------------------------
>
> Key: DERBY-3446
> URL: https://issues.apache.org/jira/browse/DERBY-3446
> Project: Derby
> Issue Type: Task
> Components: JDBC, Network Client
> Affects Versions: 10.4.1.3
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.4.1.4, 10.5.0.0
>
> Attachments: derby-3446-1a_rs_getstatement.diff, derby-3446-1a_rs_getstatement.stat, derby-3446-2a_rs_getstatement_alternative.diff, derby-3446-2a_rs_getstatement_alternative.stat, derby-3446-2b_rs_getstatement_alternative.diff, derby-3446-2c_rs_getstatement_alternative.diff, derby-3446-3a-stmtpool_test.diff
>
>
> ResultSet.getStatement must return the correct statement, that is the statement that created the result set.
> It is particularly important for result set created by logical statements, as leaking of physical statements can cause all kinds of side effects in a connection pooling environment.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (DERBY-3446) Make ResultSet.getStatemet return the
correct statement when created by a logical statement
Posted by "Kristian Waagan (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-3446?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kristian Waagan updated DERBY-3446:
-----------------------------------
Attachment: derby-3446-1a_rs_getstatement.stat
derby-3446-1a_rs_getstatement.diff
'derby-3446-1a_rs_getstatement.diff' is a proposal of how to make ResultSet.getStatement return the correct statement when using logical statements to create the result set. The changes are pretty isolated.
I think there are alternative approaches, but when I tried one of them it caused massive code changes to classes in both am and net. I observe that the issue is handled in a similar way as in the patch in the embedded driver.
If you want to run the test with and without the implementing code, remember to also apply the patch for DERBY-3329.
> Make ResultSet.getStatemet return the correct statement when created by a logical statement
> -------------------------------------------------------------------------------------------
>
> Key: DERBY-3446
> URL: https://issues.apache.org/jira/browse/DERBY-3446
> Project: Derby
> Issue Type: Task
> Components: JDBC, Network Client
> Affects Versions: 10.4.0.0
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.4.0.0
>
> Attachments: derby-3446-1a_rs_getstatement.diff, derby-3446-1a_rs_getstatement.stat
>
>
> ResultSet.getStatement must return the correct statement, that is the statement that created the result set.
> It is particularly important for result set created by logical statements, as leaking of physical statements can cause all kinds of side effects in a connection pooling environment.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (DERBY-3446) Make ResultSet.getStatemet return the
correct statement when created by a logical statement
Posted by "Kristian Waagan (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-3446?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kristian Waagan updated DERBY-3446:
-----------------------------------
Attachment: derby-3446-3a-stmtpool_test.diff
'derby-3446-3a-stmtpool_test.diff' adds the two tests to jdbcapi.StatementPoolingTest. The case of non-statement pooling environments is already tested in several other tests.
I also tried running the two new tests in a XA environment, and they passed.
Patch ready for review.
> Make ResultSet.getStatemet return the correct statement when created by a logical statement
> -------------------------------------------------------------------------------------------
>
> Key: DERBY-3446
> URL: https://issues.apache.org/jira/browse/DERBY-3446
> Project: Derby
> Issue Type: Task
> Components: JDBC, Network Client
> Affects Versions: 10.4.1.3
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.4.1.3
>
> Attachments: derby-3446-1a_rs_getstatement.diff, derby-3446-1a_rs_getstatement.stat, derby-3446-2a_rs_getstatement_alternative.diff, derby-3446-2a_rs_getstatement_alternative.stat, derby-3446-2b_rs_getstatement_alternative.diff, derby-3446-2c_rs_getstatement_alternative.diff, derby-3446-3a-stmtpool_test.diff
>
>
> ResultSet.getStatement must return the correct statement, that is the statement that created the result set.
> It is particularly important for result set created by logical statements, as leaking of physical statements can cause all kinds of side effects in a connection pooling environment.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (DERBY-3446) Make ResultSet.getStatemet return
the correct statement when created by a logical statement
Posted by "Knut Anders Hatlen (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-3446?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12571398#action_12571398 ]
Knut Anders Hatlen commented on DERBY-3446:
-------------------------------------------
+1 to 2b
I think you are right that it would make sense to refer to the PreparedStatements as am.PreparedStatement. This cache is probably going to be tied to the client driver anyway, and in the client driver you don't get more generic than the am ('Abstract Machine') package... And as long as you need to cast to am.PreparedStatement, it's probably better to declare them as am.PreparedStatement (if nothing else, avoiding the casts gives better compile-time error checking).
> Make ResultSet.getStatemet return the correct statement when created by a logical statement
> -------------------------------------------------------------------------------------------
>
> Key: DERBY-3446
> URL: https://issues.apache.org/jira/browse/DERBY-3446
> Project: Derby
> Issue Type: Task
> Components: JDBC, Network Client
> Affects Versions: 10.4.0.0
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.4.0.0
>
> Attachments: derby-3446-1a_rs_getstatement.diff, derby-3446-1a_rs_getstatement.stat, derby-3446-2a_rs_getstatement_alternative.diff, derby-3446-2a_rs_getstatement_alternative.stat, derby-3446-2b_rs_getstatement_alternative.diff
>
>
> ResultSet.getStatement must return the correct statement, that is the statement that created the result set.
> It is particularly important for result set created by logical statements, as leaking of physical statements can cause all kinds of side effects in a connection pooling environment.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Resolved: (DERBY-3446) Make ResultSet.getStatement return
the correct statement when created by a logical statement
Posted by "Kristian Waagan (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-3446?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kristian Waagan resolved DERBY-3446.
------------------------------------
Resolution: Fixed
> Make ResultSet.getStatement return the correct statement when created by a logical statement
> --------------------------------------------------------------------------------------------
>
> Key: DERBY-3446
> URL: https://issues.apache.org/jira/browse/DERBY-3446
> Project: Derby
> Issue Type: Task
> Components: JDBC, Network Client
> Affects Versions: 10.4.1.3
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.4.1.4, 10.5.0.0
>
> Attachments: derby-3446-1a_rs_getstatement.diff, derby-3446-1a_rs_getstatement.stat, derby-3446-2a_rs_getstatement_alternative.diff, derby-3446-2a_rs_getstatement_alternative.stat, derby-3446-2b_rs_getstatement_alternative.diff, derby-3446-2c_rs_getstatement_alternative.diff, derby-3446-3a-stmtpool_test.diff
>
>
> ResultSet.getStatement must return the correct statement, that is the statement that created the result set.
> It is particularly important for result set created by logical statements, as leaking of physical statements can cause all kinds of side effects in a connection pooling environment.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (DERBY-3446) Make ResultSet.getStatemet return the
correct statement when created by a logical statement
Posted by "Kristian Waagan (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-3446?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kristian Waagan updated DERBY-3446:
-----------------------------------
Derby Info: [Patch Available]
Patch 1a ready for review.
> Make ResultSet.getStatemet return the correct statement when created by a logical statement
> -------------------------------------------------------------------------------------------
>
> Key: DERBY-3446
> URL: https://issues.apache.org/jira/browse/DERBY-3446
> Project: Derby
> Issue Type: Task
> Components: JDBC, Network Client
> Affects Versions: 10.4.0.0
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.4.0.0
>
> Attachments: derby-3446-1a_rs_getstatement.diff, derby-3446-1a_rs_getstatement.stat
>
>
> ResultSet.getStatement must return the correct statement, that is the statement that created the result set.
> It is particularly important for result set created by logical statements, as leaking of physical statements can cause all kinds of side effects in a connection pooling environment.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (DERBY-3446) Make ResultSet.getStatemet return the
correct statement when created by a logical statement
Posted by "Kristian Waagan (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/DERBY-3446?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kristian Waagan updated DERBY-3446:
-----------------------------------
Attachment: derby-3446-2a_rs_getstatement_alternative.stat
derby-3446-2a_rs_getstatement_alternative.diff
Thanks for looking at the patch Knut Anders.
I think you are correct about the volatile, I will remove it if we end with that approach.
I like the alternative approach, but I couldn't implement it exactly as you suggest. Have I misunderstood?
LogicalStatementEntity does not implement java.sql.Statement, and can't be set as the owner. A solution would be to move the setting of the ownership into the subclasses, which would currently also only need to happen once due to the class hierarchy (in LogicalPreparedStatement).
The other thing that changes, is that the physical statement now has a reference to the logical statement. This could stop the most recently used logical statement from being garbage collected, which is not a big deal I believe. However, I'm wondering if there are other scenarios where this reference could cause side effects / problems?
I added a reset of the owner to LogicalStatementEntity.close.
I also moved the tests to lang.ResultSetsFromPreparedStatementTest, to have the code tested for other JVM versions than Java SE 6 as well.
Have a look at patch (2a) and comment / change it if you want to. I think I like approach 2 better.
> Make ResultSet.getStatemet return the correct statement when created by a logical statement
> -------------------------------------------------------------------------------------------
>
> Key: DERBY-3446
> URL: https://issues.apache.org/jira/browse/DERBY-3446
> Project: Derby
> Issue Type: Task
> Components: JDBC, Network Client
> Affects Versions: 10.4.0.0
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.4.0.0
>
> Attachments: derby-3446-1a_rs_getstatement.diff, derby-3446-1a_rs_getstatement.stat, derby-3446-2a_rs_getstatement_alternative.diff, derby-3446-2a_rs_getstatement_alternative.stat
>
>
> ResultSet.getStatement must return the correct statement, that is the statement that created the result set.
> It is particularly important for result set created by logical statements, as leaking of physical statements can cause all kinds of side effects in a connection pooling environment.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.