Skip to content

Bump redis from 2.10.6 to 3.4.1 #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
May 19, 2020
Merged

Conversation

dependabot-preview[bot]
Copy link
Contributor

@dependabot-preview dependabot-preview bot commented Mar 24, 2020

Bumps redis from 2.10.6 to 3.4.1.

Changelog

Sourced from redis's changelog.

  • 3.4.1
    • Move the username argument in the Redis and Connection classes to the end of the argument list. This helps those poor souls that specify all their connection options as non-keyword arguments. #1276
    • Prior to ACL support, redis-py ignored the username component of Connection URLs. With ACL support, usernames are no longer ignored and are used to authenticate against an ACL rule. Some cloud vendors with managed Redis instances (like Heroku) provide connection URLs with a username component pre-ACL that is not intended to be used. Sending that username to Redis servers < 6.0.0 results in an error. Attempt to detect this condition and retry the AUTH command with only the password such that authentication continues to work for these users. #1274
    • Removed the eq hooks to Redis and ConnectionPool that were added in 3.4.0. This ended up being a bad idea as two separate connection pools be considered equal yet manage a completely separate set of connections.
  • 3.4.0
    • Allow empty pipelines to be executed if there are WATCHed keys. This is a convenient way to test if any of the watched keys changed without actually running any other commands. Thanks @brianmaissy. #1233, #1234
    • Removed support for end of life Python 3.4.
    • Added support for all ACL commands in Redis 6. Thanks @IAmATeaPot418 for helping.
    • Pipeline instances now always evaluate to True. Prior to this change, pipeline instances relied on len for boolean evaluation which meant that pipelines with no commands on the stack would be considered False. #994
    • Client instances and Connection pools now support a 'client_name' argument. If supplied, all connections created will call CLIENT SETNAME as soon as the connection is opened. Thanks to @Habbie for supplying the basis of this change. #802
    • Added the 'ssl_check_hostname' argument to specify whether SSL connections should require the server hostname to match the hostname specified in the SSL cert. By default 'ssl_check_hostname' is False for backwards compatibility. #1196
    • Slightly optimized command packing. Thanks @Deneby67. #1255
    • Added support for the TYPE argument to SCAN. Thanks @netocp. #1220
    • Better thread and fork safety in ConnectionPool and BlockingConnectionPool. Added better locking to synchronize critical sections rather than relying on CPython-specific implementation details relating to atomic operations. Adjusted how the pools identify and deal with a fork. Added a ChildDeadlockedError exception that is raised by child processes in the very unlikely chance that a deadlock is encountered. Thanks @gmbnomis, @mdellweg, @yht804421715. #1270, #1138, #1178, #906, #1262
    • Added eq hooks to the Redis and ConnectionPool classes. Thanks @brainix. #1240
  • 3.3.11
    • Further fix for the SSLError -> TimeoutError mapping to work
... (truncated)
Commits
  • 4b7f562 3.4.1
  • 1b05075 remove Redis and ConnectionPool eq comparison
  • 5a1f3c4 Move the username argument in the Redis and Connection classes to the end
  • e39ae45 Provide AUTH fallback support for connection URLs with a username component
  • ab1c659 3.4.0
  • 4287963 better thread-safety for ConnectionPool (#1270)
  • 09a17ea Fix spelling in docstring (#1272)
  • a77ad50 add type filter to scan function
  • 272d313 Slight optimization to command packing.
  • 9cbb48a Add test for pipeline.transaction(value_from_callable=True)
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Note: This repo was added to Dependabot recently, so you'll receive a maximum of 3 PRs for your first few update runs. Once an update run creates fewer than 3 PRs we'll remove that limit.

You can always request more updates by clicking Bump now in your Dependabot dashboard.

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language
  • @dependabot badge me will comment on this PR with code to add a "Dependabot enabled" badge to your readme

Additionally, you can set the following in your Dependabot dashboard:

  • Update frequency (including time of day and day of week)
  • Pull request limits (per update run and/or open at any time)
  • Out-of-range updates (receive only lockfile updates, if desired)
  • Security updates (receive only security updates, if desired)

@dependabot-preview dependabot-preview bot added the dependencies Pull requests that update a dependency file label Mar 24, 2020
@dependabot-preview dependabot-preview bot force-pushed the dependabot/pip/redis-3.4.1 branch from 1ef0903 to 11a981e Compare March 25, 2020 12:13
@i-trofimtschuk i-trofimtschuk requested review from a team and kaloyanm and removed request for a team and kaloyanm March 25, 2020 12:21
@dependabot-preview dependabot-preview bot force-pushed the dependabot/pip/redis-3.4.1 branch from 11a981e to e90d89d Compare March 26, 2020 10:07
Co-Authored-By: Tatiana Vasilevskaya <[email protected]>
@vasilty vasilty force-pushed the dependabot/pip/redis-3.4.1 branch from 4f68e9a to 3f7929d Compare March 26, 2020 12:36
@vasilty vasilty force-pushed the dependabot/pip/redis-3.4.1 branch from 3f7929d to e51e317 Compare March 26, 2020 12:41
@vasilty vasilty requested a review from i-trofimtschuk March 26, 2020 12:45
@@ -114,12 +114,12 @@ def get_dead_ids(self):
self.key, '-inf', oldest_valid))


worker_registry = WorkerRegistry()
worker_registry = LazyObject(lambda: WorkerRegistry())
Copy link
Member

Choose a reason for hiding this comment

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

we dont need a lambda here, right?

Suggested change
worker_registry = LazyObject(lambda: WorkerRegistry())
worker_registry = LazyObject(WorkerRegistry)

Copy link
Member

Choose a reason for hiding this comment

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

right, we could do like so. Thought that that way would be analogous to the rest, but maybe that is not better

@@ -133,4 +133,4 @@ def remove(self, queue, *, pipeline):
pipeline.srem(self.key, queue.name)


queue_registry = QueueRegistry()
queue_registry = LazyObject(lambda: QueueRegistry())
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
queue_registry = LazyObject(lambda: QueueRegistry())
queue_registry = LazyObject(QueueRegistry)

@vasilty vasilty requested a review from Chronial March 27, 2020 11:06
@dependabot-preview
Copy link
Contributor Author

A newer version of redis exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged.

@vasilty vasilty requested review from i-trofimtschuk and kaloyanm and removed request for Chronial May 18, 2020 12:19
@vasilty vasilty merged commit 1231a8e into master May 19, 2020
@vasilty vasilty deleted the dependabot/pip/redis-3.4.1 branch May 19, 2020 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants