New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WFLY-30 SFSB containing injected DataSource fails to passivate/serialize #6925
Conversation
Linux Build 5432 is now running using a merge of 1be82b8 |
Windows Build 535 outcome was FAILURE using a merge of 1be82b8 Build problems:Failed tests detected Failed tests
|
Linux Build 5432 outcome was FAILURE using a merge of 1be82b8 Build problems:Failed tests detected Failed tests
|
Linux Build 5442 is now running using a merge of 604220a |
Linux Build 5442 outcome was SUCCESS using a merge of 7ddd1b4 |
Linux Build 5443 is now running using a merge of 7ddd1b4 |
Linux Build 5443 outcome was SUCCESS using a merge of 7ddd1b4 |
@@ -134,10 +136,10 @@ public synchronized void start(StartContext startContext) throws StartException | |||
if (deploymentMD.getCfs().length != 1) { | |||
throw ConnectorLogger.ROOT_LOGGER.cannotStartDs(); | |||
} | |||
sqlDataSource = (javax.sql.DataSource) deploymentMD.getCfs()[0]; | |||
sqlDataSource = new WildFlyDataSource((javax.sql.DataSource) deploymentMD.getCfs()[0], jndiName); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there any chance use code could rely on the datasource being assignable to some other class?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should not
Can you additionally validate this fix by enabling this test? |
Linux Build 5463 is now running using a merge of 0d11edd |
Linux Build 5463 outcome was SUCCESS using a merge of 0d11edd |
@pferraro done |
WFLY-30 SFSB containing injected DataSource fails to passivate/serialize
No description provided.