googleapis / ruby-spanner-activerecord

Spanner<->ActiveRecord adapter for Ruby
MIT License
87 stars 30 forks source link

fix: _insert_record failed for other adapters #298

Closed olavloite closed 8 months ago

olavloite commented 8 months ago

The _insert_record method would fail for adapters other than Spanner, as it would call super with two arguments. This is now changed so the method checks whether the current connection uses Spanner or some other adapter, and based on that calls the super method with 1 or 2 arguments.

Fixes #292 Also fixes the issue mentioned in #294