You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Can this functionality please be added to 6.1?
I tried to fork the repo and add this but I got a lot of failing test cases, even before I changed anything 😢
Thanks
The text was updated successfully, but these errors were encountered:
@harelsafra there are indeed failing tests around this (see #333). I'm working on it now. I don't feel safe backporting this knowing that there are some unresolved bugs.
Hi,
We migrated to CRDB and are getting sporadic PG::TRSerializationFailure errors, currently using the PG driver.
We're using rails 6.1.7.3 and so need to use the 6.1 gem, and it looks like it doesn't catch and retry PG::TRSerializationFailure errors
https://github.com/cockroachdb/activerecord-cockroachdb-adapter/blob/6-1-stable/lib/active_record/connection_adapters/cockroachdb/transaction_manager.rb
While the master branch does have this handling
activerecord-cockroachdb-adapter/lib/active_record/connection_adapters/cockroachdb/transaction_manager.rb
Line 58 in 0f9a418
Can this functionality please be added to 6.1?
I tried to fork the repo and add this but I got a lot of failing test cases, even before I changed anything 😢
Thanks
The text was updated successfully, but these errors were encountered: