Bump aHash to v0.8.6 #966
Merged
Bump aHash to v0.8.6 #966
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Hi, can you explain the reason for the breaking changes? |
|
Sure! The first breaking change here is the bump from v0.7 to v0.8 itself. That's done to simply stay up-to-date with the newest release and to include a fix for their use of AES instructions on The second breaking change is the removal of the
So technically it's only one breaking change, but for consistency's sake, this PR is making two. If the second change (the removal of the hash state parameter) isn't desired, I could probably just implement that with a fold over the iterator |
|
Thanks for the explanation, which certainly sounds reasonable to me. We still need to figure out our next release and whether it's breaking, so I'm going to leave this un-merged for now. @aumetra do you mind updating the version specified to a non-yanked version? |
jaymell
approved these changes
Nov 17, 2023
barshaul
pushed a commit
to amazon-contributing/redis-rs
that referenced
this pull request
Dec 7, 2023
* Disable JSON module tests for redis 6.2.4. (redis-rs#980) This happens due to a consistent crash - RedisJSON/RedisJSON#1124 * Implement `FromRedisValue` for `Box<[T]>` and `Arc<[T]>` (redis-rs#799) * Update minimal rust version to 1.6.5 (redis-rs#982) It's required by regex v1.10.2. https://github.com/redis-rs/redis-rs/actions/runs/6760188489/job/18373652560?pr=981 * Support Mutual TLS (redis-rs#858) Add support for mutual TLS. Enable specifying client certificate and key and (optional) root certificate. * changing timeouts from usize and isize to f64 (redis-rs#988) Co-authored-by: Eythor Helgason <eytorhelgason@gmail.com> * Accept iterator at `ClusterClient` initialization (redis-rs#987) * Prefer routing to primary in a transaction. (redis-rs#986) * Rename route_pipeline to route_for_pipeline. There are 2 `route_pipeline` functions in the file, so we change one to avoid confusion. * Prefer routing to primary in a transaction. * CrossSlot error on pipeline with many routes. * Sync Pub/Sub - cache received pub/sub messages. (redis-rs#910) When sending subscibre/unsubscribe messages to the server, the pub/sub client might receive messages that it can mistake to be the response, and not return to the next `get_message` call. This fix ensures that pub/sub messages will be queued until the next `get_message` call, so that no message will be missed. * Fix sync cluster behavior with transactions. (redis-rs#983) The leading MULTI prevents the transaction from being properly routed. In order to route the request correctly, we drop the leading MULTI from the routable. * Release redis 0.23.4 / redis-test 0.2.4 (redis-rs#992) * Tests: Add retries to test cluster creation (redis-rs#994) * Revert redis-test versioning changes (redis-rs#993) Publishing to crates.io without the version specification is currently not possible. Reverting these changes for now until we have a better solution/workaround. * Fix features for `load_native_certs`. (redis-rs#996) The directive now matches the directives wrapping `load_native_certs`'s `use` statements. This would fail compilation for anyone accidentally using the "tls" feature with rustls. * Bump aHash to v0.8.6 (redis-rs#966) * Update Command expiration values to be an appropriate type (redis-rs#589) This change updates expiration values provided to some commands in the `Command` trait to be of type `i64` or `u64` where appropriate, replacing the previously used `usize` type. More specifically, the commands that are updated are - SETEX, PSETEX - Expiration value changed from `usize` to `u64` - EXPIRE, PEXPIRE, EXPIREAT, PEXPIREAT - Expiration value changed from `usize` to `i64` For the `*SETEX` commands, there is no mention of supporting negative values. And indeed, providing a negative value through redis-cli returns an error message. For the `*EXPIRE*` commands, I couldn't find the paragraph in the docs referenced in redis-rs#575, nor any other mention that the commands accept a negative expiration argument. However, in testing with redis-cli it seems a negative expiration value is valid and in checking the code I found it to confirm these observations. See [here][code1] and [here][code2]. Note that this is a breaking change. [code1]: https://github.com/redis/redis/blob/e3ef73dc2a557232c60c732705e8e6ff2050eba9/src/expire.c#L570-L571 [code2]: https://github.com/redis/redis/blob/e3ef73dc2a557232c60c732705e8e6ff2050eba9/src/expire.c#L483-L491 * Fix StreamId::contains_key signature (redis-rs#783) * Order in usage of ClusterParams. (redis-rs#997) * Move `tls_params` into `ClusterParams`. The two objects are used together, so it makes sense to join them. * Cluster: Contain `ClusterParams` internally. This matches the async cluster's structure, and ensures that fields that are added to `ClusterParams` are automatically added to the sync cluster * Release redis 0.24.0 / redis-test 0.3.0 (redis-rs#998) --------- Co-authored-by: Jacob O'Toole <jacob@jotpot.co.uk> Co-authored-by: spaceangel <129979014+sp-angel@users.noreply.github.com> Co-authored-by: eythorhel19 <70693645+eythorhel19@users.noreply.github.com> Co-authored-by: Eythor Helgason <eytorhelgason@gmail.com> Co-authored-by: Ruan Petterson <ruan@petterson.eng.br> Co-authored-by: James Lucas <jaymell@users.noreply.github.com> Co-authored-by: aumetra <aumetra@cryptolab.net> Co-authored-by: Josh Leeb-du Toit <mail@joshleeb.com> Co-authored-by: Ayush <ayushsingh1325@gmail.com>
barshaul
pushed a commit
to barshaul/redis-rs
that referenced
this pull request
Jul 11, 2024
* Fix a file. Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com> * Add CI check. Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com> --------- Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a breaking change, bumping the
ahashdependency in a semver-incompatible way