pennylane-hq / activerecord-adapter-redshift

Other
6 stars 27 forks source link

Redshift 7.1 Adapter: move code from reconnect! to reconnect #23

Closed jerryclinesmith closed 7 months ago

jerryclinesmith commented 7 months ago

Closes #19

quentindemetz commented 7 months ago

Thanks!

jerryclinesmith commented 7 months ago

Thank you for the quick merge, but you might want to rollback this change. While all of my tests, and the original test condition now pass, when running in production using multiple threads I'm getting a hard crash in Ruby. Still investigating, hoping to have an update to this Pull Request, but wanted to let you know.