Skip to content

feat: support Rails 7.1#278

Merged
olavloite merged 8 commits intomainfrom
rails-7-1
Dec 13, 2023
Merged

feat: support Rails 7.1#278
olavloite merged 8 commits intomainfrom
rails-7-1

Conversation

@olavloite
Copy link
Collaborator

@olavloite olavloite commented Nov 28, 2023

Updates the Spanner Ruby ActiveRecord provider to be compatible with Rails 7.1, but with one major exception:

  1. Rails 7.1 introduced support for composite primary keys.
  2. This makes the third-party composite primary keys gem incompatible with Rails 7.1
  3. This provider used the third-party composite primary keys gem to support interleaved tables

The above means that this PR removes support for interleaved tables when used with Rails 7.1. It does not affect anyone using Rails 7.0, 6.1 or 6.0.

We will re-introduce support for interleaved tables in a separate PR for Rails 7.1 based on the built-in composite primary key support.

@product-auto-label product-auto-label bot added the api: spanner Issues related to the googleapis/ruby-spanner-activerecord API. label Nov 28, 2023
@olavloite olavloite marked this pull request as ready for review December 1, 2023 15:15
@olavloite olavloite requested a review from a team as a code owner December 1, 2023 15:15
@olavloite olavloite requested a review from hengfengli December 1, 2023 15:28
Copy link
Collaborator

@hengfengli hengfengli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@olavloite olavloite merged commit b1785a1 into main Dec 13, 2023
@olavloite olavloite deleted the rails-7-1 branch December 13, 2023 14:07
Connorhd added a commit to Connorhd/ruby-spanner-activerecord that referenced this pull request Jan 12, 2024
This method changed to have 2 arguments in Rails 7.1, and changes in googleapis#278 to support that cause this to error on Rails 7.0. Call super with the appropriate arguments on older versions of Rails.
Connorhd added a commit to Connorhd/ruby-spanner-activerecord that referenced this pull request Jan 12, 2024
This method changed to have 2 arguments in Rails 7.1, and changes in googleapis#278 to support that cause this to error on Rails 7.0. Call super with the appropriate arguments on older versions of Rails.
Connorhd added a commit to Connorhd/ruby-spanner-activerecord that referenced this pull request Jan 12, 2024
This method changed to have 2 arguments in Rails 7.1, and changes in googleapis#278 to support that cause this to error on Rails 7.0. Call super with the appropriate arguments on older versions of Rails.
Connorhd added a commit to Connorhd/ruby-spanner-activerecord that referenced this pull request Jan 12, 2024
This method changed to have 2 arguments in Rails 7.1, and changes in googleapis#278 to support that cause this to error on Rails 7.0. Call super with the appropriate arguments on older versions of Rails.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: spanner Issues related to the googleapis/ruby-spanner-activerecord API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants