-
Notifications
You must be signed in to change notification settings - Fork 9
ER_NO_TRANSACTION on tarantool-2.2+ #53
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
Labels
bug
Something isn't working
Comments
Hello,
|
It is maybe related to #52 (I didn't investigate). |
LeonidVas
added a commit
that referenced
this issue
May 13, 2020
Before introducing a txn memory region box_txn_alloc() allocates memory using the fiber.gc region and can be used without box_txn_begin(). Now the similar trick doesn't work (a transaction must be started). To allocate memory on region without start a transaction, the region was added to struct memcached_connection. Fixes #53
The fixes for Travis failed jobs(not related with this fix): |
LeonidVas
added a commit
that referenced
this issue
May 14, 2020
Before introducing a txn memory region box_txn_alloc() allocates memory using the fiber.gc region and can be used without box_txn_begin(). Now the similar trick doesn't work (a transaction must be started). To allocate memory on region without start a transaction, the region was added to struct memcached_connection. Fixes #53
Totktonada
pushed a commit
that referenced
this issue
Jun 1, 2020
Before introducing a txn memory region box_txn_alloc() allocates memory using the fiber.gc region and can be used without box_txn_begin(). Now the similar trick doesn't work (a transaction must be started). To allocate memory on region without start a transaction, the region was added to struct memcached_connection. Fixes #53
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When trying to do
get
:It is ER_NO_TRANSACTION.
Before splitting transaction and fiber memory regions,
box_txn_alloc()
may be called w/o active transaction.Reported by Andrey Yanda.
The text was updated successfully, but these errors were encountered: