-
Notifications
You must be signed in to change notification settings - Fork 202
Release NativeLink v0.7.8 #2075
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
Release NativeLink v0.7.8 #2075
Conversation
nativelink-worker/BUILD.bazel
Outdated
| "@crates//:bytes", | ||
| "@crates//:futures", | ||
| "@crates//:hyper-1.7.0", | ||
| "@crates//:hyper-1.8.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change might be causing the CI failures
| @@ -3,187 +3,51 @@ | |||
|
|
|||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CHANGELOG is messed up. Please fix this such that that this doesn't update the previous versions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like 0.7.7 is also screwed up in the changelog
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you for the review.
acc2d85 to
feab737
Compare
feab737 to
6cf6aed
Compare
|
|
||
| [[package]] | ||
| name = "aho-corasick" | ||
| version = "1.1.3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cargo lock should be generated once again
6cf6aed to
5622ff2
Compare
…#2069) * fix: use wildcard query when Redis index value is empty When searching Redis with an empty filter, the query was generating `@nl_state:{ }` which causes a syntax error at offset 10. This fix uses `*` (match all) when the sanitized field is empty. Also updates the test mock to handle wildcard queries and skip entries without version fields (e.g., those created via HSET). Fixes: Syntax error at offset 10 near state * complete merge to comply with CI * fix: resolve merge conflict in search_by_index_prefix Merged main into fix-redis-query-syntax branch and resolved conflict in the search_by_index_prefix method to work with RecoverablePool. Changes: - Extract Client from ClientWithPermit using client.client.clone() - Pass sanitized_field as parameter to closure for proper ownership - Maintain wildcard query fix (*) for empty filter values - Maintain input sanitization security ,
5622ff2 to
d8485c4
Compare
Description
prepare
0.7.8release.Fixes #2074
Type of change
Please delete options that aren't relevant.
How Has This Been Tested?
This release has been tested with large scale workloads across a number of clients and use cases.
Checklist
bazel test //...passes locallygit amendsee some docsThis change is