Navigation Menu

Skip to content
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

Review and revise FBXADataSource implementation [JDBC178] #225

Closed
firebird-automations opened this issue Jul 30, 2011 · 8 comments
Closed

Comments

@firebird-automations
Copy link

Submitted by: @mrotteveel

Jira_subtask_inward JDBC176

Ascertain correctness of org.firebirdsql.jca.FBXADataSource and fix problems as needed.

Commits: b8b9fc7 8b38116

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

assignee: Roman Rokytskyy [ rrokytskyy ] => Mark Rotteveel [ avalanche1979 ]

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

Fix Version: Jaybird 2.2 [ 10053 ]

Component: JCA layer [ 10052 ]

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

status: Open [ 1 ] => In Progress [ 3 ]

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

description: Ascertain correctness of org.firebirdsql.FBXADataSource and fix problems as needed. => Ascertain correctness of org.firebirdsql.jca.FBXADataSource and fix problems as needed.

@firebird-automations
Copy link
Author

Commented by: @mrotteveel

Opted to reimplement it based on org.firebirdsql.ds.FBConnectionPoolDataSource. I still need to review some of the aspects of XA and write tests, but I committed the current implementation

@firebird-automations
Copy link
Author

Commented by: @mrotteveel

Added test, made some more improvements to handling of a close of the logical Connection created from the XAConnection

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

status: In Progress [ 3 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

status: Resolved [ 5 ] => Closed [ 6 ]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants