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

64-bit transaction IDs are not stored properly in status vector [CORE6110] #6359

Closed
firebird-issue-importer opened this issue Jul 29, 2019 · 5 comments

Comments

@firebird-issue-importer

Submitted by: @samofatov

Assigned to: @ilya071294

Transaction ID type (TraNumber) has become 64-bit in Firebird 3.0 and later versions.
isc_concurrent_transaction and isc_rec_in_limbo errors report concurrent transaction ID as isc_arg_number (32-bit signed integer).
This results in transaction numbers truncation in 32-bit builds or when status vector is sent over the wire.

I suggest to change transaction ID type to isc_arg_string when it is stored in status vector.

Commits: 199b0a6 641eca3 52ab84e 696673b

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jan 28, 2020

Modified by: @samofatov

description: Transaction ID type (TraNumber) has become 64-bit in Firebird 3.0 and later versions.
isc_concurrent_transaction reports concurrent transaction ID as isc_arg_number (32-bit signed integer).
This results in transaction numbers truncation in 32-bit builds or when status vector is sent over the wire.

I suggest to change transaction ID type to isc_arg_string when it is stored in status vector.

=>

Transaction ID type (TraNumber) has become 64-bit in Firebird 3.0 and later versions.
isc_concurrent_transaction and isc_rec_in_limbo errors report concurrent transaction ID as isc_arg_number (32-bit signed integer).
This results in transaction numbers truncation in 32-bit builds or when status vector is sent over the wire.

I suggest to change transaction ID type to isc_arg_string when it is stored in status vector.

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Feb 7, 2020

Modified by: @AlexPeshkoff

assignee: Ilya Eremin [ ilya071294 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Feb 7, 2020

Modified by: @AlexPeshkoff

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

Fix Version: 4.0 Beta 2 [ 10888 ]

Fix Version: 3.0.6 [ 10889 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Mar 20, 2020

Modified by: @pavel-zotov

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

QA Status: No test => Cannot be tested

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Mar 20, 2020

Modified by: @pavel-zotov

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

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