Skip to content

Commit

Permalink
Update regex and globset to pull in memory usage fixes
Browse files Browse the repository at this point in the history
Summary:
BurntSushi has fixed a memory usage regression introduced by regex 1.9 which caused Buck to allocate and retain significantly more memory when using moderately sized `buck2_common::ignores::ignore_set::IgnoreSet` objects concurrently from many threads.

- Bug report: **[rust-lang/regex#1059](rust-lang/regex#1059) *"1.9 memory usage: globset-generated RegexSet allocates and retains 48× more memory (600MB) vs regex 1.8"***

- Globset fix: **[BurntSushi/ripgrep#25770](https://github.com/BurntSushi/ripgrep/pull/25770) *"globset: use non-capture groups in regex transform"***

- Regex fix: **[rust-lang/regex#1062](rust-lang/regex#1062) *"fix memory usage regression for RegexSet with capture groups"***

Reviewed By: zertosh

Differential Revision: D48095372

fbshipit-source-id: ec11c2bcaccbd26354d6d0a0398000134eaf3681
  • Loading branch information
David Tolnay authored and facebook-github-bot committed Aug 5, 2023
1 parent a86e49a commit bcad302
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion shim/third-party/rust/Cargo.toml
Expand Up @@ -26,7 +26,7 @@ md-5 = "0.10"
once_cell = "1.12"
proc-macro2 = { version = "1.0.64", features = ["span-locations"] }
quote = "1.0.29"
regex = "1.6.0"
regex = "1.9.2"
rustc-hash = "1.1.0"
serde = { version = "1.0.176", features = ["derive", "rc"] }
similar = { version = "2.2.0", features = ["inline"] }
Expand Down
2 changes: 1 addition & 1 deletion signed_source/Cargo.toml
Expand Up @@ -17,5 +17,5 @@ bstr = { version = "1.4.0", features = ["serde", "std", "unicode"] }
hex = "0.4.3"
md-5 = "0.10"
once_cell = "1.12"
regex = "1.6.0"
regex = "1.9.2"
thiserror = "1.0.43"

0 comments on commit bcad302

Please sign in to comment.