activerecord-cockroachdb-adapter
activerecord-cockroachdb-adapter copied to clipboard
Request to add PG::TRSerializationFailure handling to 6-1-stable
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 https://github.com/cockroachdb/activerecord-cockroachdb-adapter/blob/0f9a4181080e5b1ae2040fc91c5b0d891c248505/lib/active_record/connection_adapters/cockroachdb/transaction_manager.rb#L58
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