Issue Details (XML | Word | Printable)

Key: DNET-251
Type: Sub-task Sub-task
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Jiri Cincura
Reporter: Jiri Cincura
Votes: 2
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
.NET Data provider
DNET-247

FbConnectionInternal public FbTransaction BeginTransaction methods issue

Created: 04/Jul/09 03:59 PM   Updated: 04/Nov/17 02:56 PM
Component/s: ADO.NET Provider
Affects Version/s: None
Fix Version/s: 5.12.0.0


 Description  « Hide
class FbConnectionInternal
public FbTransaction BeginTransaction methods
as I think, if this.activeTransaction.BeginTransaction() call raises exception then activeTransaction should be disposed and nulled. if not - next time it will cause HasActiveTransaction to return true and InvalidOperationException("A transaction is currently active. Parallel transactions are not supported.") will be thrown. InvalidOperationException is not FbException and complecates exception handling in outer code.


 All   Comments   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Jiri Cincura made changes - 18/Jul/17 10:30 AM
Field Original Value New Value
Fix Version/s vNext [ 10833 ]
Jiri Cincura made changes - 22/Aug/17 11:04 AM
Status Open [ 1 ] In Progress [ 3 ]
Jiri Cincura made changes - 22/Aug/17 11:19 AM
Status In Progress [ 3 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Michal Catlos added a comment - 02/Nov/17 02:55 PM
In version 5.11.0.0 is this issue not fixed. _activeTransaction.BeginTransaction(options) throws FbException and not IscException, so DisposeTransaction() in catch block is never called.

Jiri Cincura added a comment - 02/Nov/17 03:16 PM
Correct. The FbTransaction methods (being internal) should not handle the IscException and wrap it. It's a task for FbConnectionInternal, where the method is public. I'll change it later this week.

Jiri Cincura made changes - 04/Nov/17 02:52 PM
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Jiri Cincura added a comment - 04/Nov/17 02:56 PM
4ef2411ae64dfd48457cb4d2edb2565ba41540a5

Jiri Cincura made changes - 04/Nov/17 02:56 PM
Status Reopened [ 4 ] Resolved [ 5 ]
Fix Version/s 5.12.0.0 [ 10854 ]
Fix Version/s 5.10.0.0 [ 10833 ]
Resolution Fixed [ 1 ]