Skip to content

Commit

Permalink
Zed update through "lake: make rangeWrapper filters safe for concurre…
Browse files Browse the repository at this point in the history
…nt use" by nwt

This is an auto-generated commit with a Zed dependency update. The Zed PR
brimdata/super#2969, authored by @nwt,
has been merged.

lake: make rangeWrapper filters safe for concurrent use

Because of concurrency in the ZNG scanner, separate filters returned by
a zbuf.Filter's methods must be safe for concurrent use in separate
goroutines.  Filters returned by lake.rangeWrapper.AsFilter do not have
this property because they share the single expr.ValueCompareFn in
rangeWrapper.compare.  Fix this by creating a new expr.ValueCompareFn
for each filter returned.

Closes brimdata/super#2960.
  • Loading branch information
Brim Automation committed Aug 27, 2021
1 parent ac680cc commit 185344d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,7 @@
"tee-1": "^0.2.0",
"tree-model": "^1.0.7",
"valid-url": "^1.0.9",
"zed": "git+https://github.com/brimdata/zed.git#5074a824b132d5788d529c82a54b0008f5ff8160"
"zed": "git+https://github.com/brimdata/zed.git#a2b7592fdd259039ee3df5979803bbb89f35bf51"
},
"optionalDependencies": {
"electron-installer-debian": "^3.0.0",
Expand Down

0 comments on commit 185344d

Please sign in to comment.