redis-rb 4.0 support #30748

jeremy · 2017-09-29T05:12:41Z

Loosen Redis version dep to >= 3.3, < 5
Bump redis-namespace for looser Redis version dep
Avoid using the underlying redis.client directly
Use Redis.new instead of Redis.connect

Pending gem releases for Sidekiq, redis-namespace, and resque-scheduler.

matthewd · 2017-09-29T07:30:27Z

actioncable/lib/action_cable/subscription_adapter/redis.rb

@@ -2,7 +2,7 @@

 require "thread"

-gem "redis", "~> 3.0"
+gem "redis", "~> 3", "< 5"


This needs to be ">= 3", "< 5"; "~> 3" and "~> 3.0" both mean the same thing.

matthewd

I don't suppose the new version has any better options for our @raw_client / flush hackery?

jeremy · 2017-10-08T21:36:33Z

Doesn't appear to.

* Use `gem 'redis', '~> 4.0'` for new app Gemfiles * Loosen Action Cable redis-rb dep to `>= 3.3, < 5` * Bump redis-namespace for looser Redis version dep * Avoid using the underlying `redis.client` directly * Use `Redis.new` instead of `Redis.connect`

eileencodes · 2017-12-07T21:10:13Z

Hey @jeremy the 5-1-stable branch is failing on travis since it auto updates the redis gem to 4.0.1. Should be be backported to the 5-1-stable branch or should 5-1-stable have a lower redis dependency requirement?

Rails master adds support for redis-rb 4.0 but when Rails 5-1-stable branch gets an updated redis there are exceptions in the travis log. ``` Error: RedisAdapterTest#test_channel_filtered_broadcast: Gem::LoadError: Specified 'redis' for Action Cable pubsub adapter, but the gem is not loaded. Add `gem 'redis'` to your Gemfile (and ensure its version is at the minimum required by Action Cable). ``` This PR backports Jeremy's work so that the build can be green. [Eileen M. Uchitelle & Jeremy Daer]

rails#30748 was backported to `5-1-stable` by 3789531. This commit adds missing changelog entry and fixes others.

- Remove ``` * Namespace error pages' CSS selectors to stop the styles from bleeding into other pages when using Turbolinks. ([Pull Request](rails#28814)) ``` since it was backported to `5-1-stable` by 50d5baf and `5-0-stable` by d1c4a39. - Remove ``` * Allow irb options to be passed from `rails console` command. ([Pull Request](rails#29010)) ``` since it was backported to `5-1-stable` by e91b483. - Remove ``` * Load environment file in `dbconsole` command. ([Pull Request](rails#29725)) ``` since it was backported to `5-1-stable` by 7f93428. - Remove ``` * Gemfile for new apps: upgrade redis-rb from ~> 3.0 to 4.0. ([Pull Request](rails#30748)) ``` since it was backported to `5-1-stable` by 3789531. - Remove ``` * Fix minitest rails plugin. The custom reporters are added only if needed. This will fix conflicts with others plugins. ([Commit](rails@ac99916)) ``` since it was backported to `5-1-stable` by caa7695. - Remove ``` * Add support for compatibility with redis-rb gem for 4.0 version. ([Pull Request](rails#30748)) ``` since it was backported to `5-1-stable` by 3789531. - Remove ``` * Add `action_controller_api` and `action_controller_base` load hooks to be called in `ActiveSupport.on_load`. ([Pull Request](rails#28402)) ``` since it was backported to `5-1-stable` by b9a5fd7. - Remove ``` * `driven_by` now registers poltergeist and capybara-webkit. ([Pull Request](rails#29315)) ``` since it was backported to `5-1-stable` by c5dd451. - Remove ``` * Fallback `ActionController::Parameters#to_s` to `Hash#to_s`. ([Pull Request](rails#29630)) ``` since it was backported to `5-1-stable` by c1014e4 and `5-0-stable` by 0e71fc3. - Remove ``` * Make `take_failed_screenshot` work within engine. ([Pull Request](rails#30421)) ``` since it was backported to `5-1-stable` by 595a231. - Remove ``` * Fix optimized url helpers when using relative url root. ([Pull Request](rails#31261)) ``` since it was backported to `5-1-stable` by e9b7795. - Remove ``` * Update `distance_of_time_in_words` helper to display better error messages for bad input. ([Pull Request](rails#20701)) ``` since it was backported to `5-1-stable` by 2c97fbf. - Remove ``` * Generate field ids in `collection_check_boxes` and `collection_radio_buttons`. ([Pull Request](rails#29412)) ``` since it was backported to `5-1-stable` by 2d8c10a. - Remove ``` * Fix issues with scopes and engine on `current_page?` method. ([Pull Request](rails#29503)) ``` since it was backported to `5-1-stable` by 2135daf. - Remove ``` * Bring back proc with arity of 1 in `ActionMailer::Base.default` proc since it was supported in Rails 5.0 but not deprecated. ([Pull Request](rails#30391)) ``` since it was backported to `5-1-stable` by b2bedb1. - Remove ``` * Add type caster to `RuntimeReflection#alias_name`. ([Pull Request](rails#28961)) ``` since it was backported to `5-1-stable` by f644e7a. - Remove ``` * Loading model schema from database is now thread-safe. ([Pull Request](rails#29216)) ``` since it was backported to `5-1-stable` by 02926cf. and `5-0-stable` by 84bcfe5 - Remove ``` * Fix destroying existing object does not work well when optimistic locking enabled and `locking_column` is null in the database. ([Pull Request](rails#28926)) ``` since it was backported to `5-1-stable` by e498052. - Remove ``` * `ActiveRecord::Persistence#touch` does not work well when optimistic locking enabled and `locking_column`, without default value, is null in the database. ([Pull Request](rails#28914)) ``` since it was backported to `5-1-stable` by 1e2f63d. - Remove ``` * Previously, when building records using a `has_many :through` association, if the child records were deleted before the parent was saved, they would still be persisted. Now, if child records are deleted before the parent is saved on a `has_many :through` association, the child records will not be persisted. ([Pull Request](rails#29593)) ``` since it was backported to `5-1-stable` by a22c39e. - Remove ``` * Query cache was unavailable when entering the `ActiveRecord::Base.cache` block without being connected. ([Pull Request](rails#29609)) ``` since it was backported to `5-1-stable` by fd6c8cd and `5-0-stable` by 9f2532b. - Remove ``` * `Relation#joins` is no longer affected by the target model's `current_scope`, with the exception of `unscoped`. ([Commit](rails@5c71000)) ``` since it was backported to `5-1-stable` by 3630d63. - Remove ``` * Fix `unscoped(where: [columns])` removing the wrong bind values. ([Pull Request](rails#29780)) ``` since it was backported to `5-1-stable` by d378fcb. - Remove ``` * When a `has_one` association is destroyed by `dependent: destroy`, `destroyed_by_association` will now be set to the reflection, matching the behaviour of `has_many` associations. ([Pull Request](rails#29855)) ``` since it was backported to `5-1-stable` by 8254a8b. - Remove ``` * Fix `COUNT(DISTINCT ...)` with `ORDER BY` and `LIMIT` to keep the existing select list. ([Pull Request](rails#29848)) ``` since it was backported to `5-1-stable` by 0e8d4ed. - Remove ``` * Ensure `sum` honors `distinct` on `has_many :through` associations. ([Commit](rails@566f1fd)) ``` since it was backported to `5-1-stable` by c0a1dc2. - Remove ``` * Fix `COUNT(DISTINCT ...)` for `GROUP BY` with `ORDER BY` and `LIMIT`. ([Commit](rails@5668dc6)) ``` since it was backported to `5-1-stable` by 87ca68e. - Remove ``` * MySQL: Don't lose `auto_increment: true` in the `db/schema.rb`. ([Commit](rails@9493d45)) ``` since it was backported to `5-1-stable` by 8b6e694. - Remove ``` * Fix longer sequence name detection for serial columns. ([Pull Request](rails#28339)) ``` since it was backported to `5-1-stable` by af9c170 and `5-0-stable` by 7025b1d. - Remove ``` * Fix `bin/rails db:setup` and `bin/rails db:test:prepare` create wrong ar_internal_metadata's data for a test database. ([Pull Request](rails#30579)) ``` since it was backported to `5-1-stable` by bb67b5f and `5-0-stable` by 60437e6. - Remove ``` * Fix conflicts `counter_cache` with `touch: true` by optimistic locking. ([Pull Request](rails#31405)) ``` since it was backported to `5-1-stable` by 5236dda. - Remove ``` * Fix `count(:all)` to correctly work `distinct` with custom SELECT list. ([Commit](rails@c6cd9a5)) ``` since it was backported to `5-1-stable` by 6beb4de. - Remove ``` * Fix to invoke callbacks when using `update_attribute`. ([Commit](rails@732aa34)) ``` since it was backported to `5-1-stable` by 6346683. - Remove ``` * Use `count(:all)` in `HasManyAssociation#count_records` to prevent invalid SQL queries for association counting. ([Pull Request](rails#27561)) ``` since it was backported to `5-1-stable` by eef3c89. - Remove ``` * Fix `count(:all)` with eager loading and having an order other than the driving table. ([Commit](rails@ebc09ed)) ``` since it was backported to `5-1-stable` by 6df9b69. - Remove ``` * PostgreSQL: Allow pg-1.0 gem to be used with Active Record. ([Pull Request](rails#31671)) ``` since it was backported to `5-1-stable` by a9c06f6. - Remove ``` * Fix that after commit callbacks on update does not triggered when optimistic locking is enabled. ([Commit](rails@7f9bd03)) ``` since it was backported to `5-1-stable` by aaee10e. - Remove ``` * Fix regression in numericality validator when comparing Decimal and Float input values with more scale than the schema. ([Pull Request](rails#28584)) ``` since it was backported to `5-1-stable` by 5b1c3e5. Note that there was incorrect link to PR, original PR is rails#29249. - Remove ``` * Fix to working before/after validation callbacks on multiple contexts. ([Pull Request](rails#31483)) ``` since it was backported to `5-1-stable` by 0f7046a. - Remove ``` * Fix implicit coercion calculations with scalars and durations. ([Pull Request](rails#29163), [Pull Request](rails#29971)) ``` since it was backported to `5-1-stable` by 51ea27c, 4d82e2a. - Remove ``` * Fix modulo operations involving durations. ([Commit](rails@a54e13b)) ``` since it was backported to `5-1-stable` by 233fa7e. - Remove ``` * Return all mappings for a timezone identifier in `country_zones`. ([Commit](rails@cdce6a7)) ``` since it was backported to `5-1-stable` by 0222ebb. - Remove ``` * Add support for compatibility with redis-rb gem for 4.0 version. ([Pull Request](rails#30748)) ``` since it was backported to `5-1-stable` by 3789531. Related to rails#32252. Related to rails#32222, rails#32222 (comment). Follow up a489cc8.

Rails master adds support for redis-rb 4.0 but when Rails 5-1-stable branch gets an updated redis there are exceptions in the travis log. ``` Error: RedisAdapterTest#test_channel_filtered_broadcast: Gem::LoadError: Specified 'redis' for Action Cable pubsub adapter, but the gem is not loaded. Add `gem 'redis'` to your Gemfile (and ensure its version is at the minimum required by Action Cable). ``` This PR backports Jeremy's work so that the build can be green. [Eileen M. Uchitelle & Jeremy Daer]

jeremy force-pushed the redis-rb-4.0.1 branch from 5cc32a4 to ff80fa3 Compare September 29, 2017 05:15

matthewd reviewed Sep 29, 2017

View reviewed changes

matthewd mentioned this pull request Sep 29, 2017

redis gem 4+ incompatibility "Specified 'redis' for Action Cable pubsub adapter, but the gem is not loaded" #30527

Closed

jeremy force-pushed the redis-rb-4.0.1 branch from ff80fa3 to cde3627 Compare September 29, 2017 22:45

jeremy mentioned this pull request Sep 30, 2017

Adapters bubble up gem version mismatches for their dependencies #30757

Merged

jeremy force-pushed the redis-rb-4.0.1 branch from cde3627 to 03268d3 Compare September 30, 2017 22:38

matthewd approved these changes Oct 6, 2017

View reviewed changes

jeremy force-pushed the redis-rb-4.0.1 branch 2 times, most recently from 9db61ed to 9298fb9 Compare October 8, 2017 22:34

redis-rb 4.0 support

53c516d

* Use `gem 'redis', '~> 4.0'` for new app Gemfiles * Loosen Action Cable redis-rb dep to `>= 3.3, < 5` * Bump redis-namespace for looser Redis version dep * Avoid using the underlying `redis.client` directly * Use `Redis.new` instead of `Redis.connect`

jeremy force-pushed the redis-rb-4.0.1 branch from 9298fb9 to 53c516d Compare October 8, 2017 22:38

jeremy added this to the 5.2.0 milestone Oct 8, 2017

jeremy merged commit 5fb2822 into rails:master Oct 8, 2017

jeremy deleted the redis-rb-4.0.1 branch October 8, 2017 23:14

kamipo mentioned this pull request Oct 9, 2017

ActionCable → upgrade to Redis 4 #30840

Closed

bogdanvlviv added a commit to bogdanvlviv/rails that referenced this pull request Mar 14, 2018

Fix changelog entries of 5-1-stable [ci skip]

5dfb3a2

rails#30748 was backported to `5-1-stable` by 3789531. This commit adds missing changelog entry and fixes others.

bogdanvlviv mentioned this pull request Mar 14, 2018

Fix changelog entries of 5-1-stable [ci skip] #32252

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

redis-rb 4.0 support #30748

redis-rb 4.0 support #30748

jeremy commented Sep 29, 2017 •

edited by matthewd

matthewd Sep 29, 2017

matthewd left a comment

jeremy commented Oct 8, 2017

eileencodes commented Dec 7, 2017

redis-rb 4.0 support #30748

redis-rb 4.0 support #30748

Conversation

jeremy commented Sep 29, 2017 • edited by matthewd

matthewd Sep 29, 2017

Choose a reason for hiding this comment

matthewd left a comment

Choose a reason for hiding this comment

jeremy commented Oct 8, 2017

eileencodes commented Dec 7, 2017

jeremy commented Sep 29, 2017 •

edited by matthewd