Skip to content

Commit

Permalink
Zed update through "remove mapstructure" by mccanne
Browse files Browse the repository at this point in the history
This is an auto-generated commit with a Zed dependency update. The Zed PR
brimdata/super#2703, authored by @mccanne,
has been merged.

remove mapstructure

I have tripped over mapstructure one too many times so this commit
removes it.  When/if the extra round-trip through JSON adds
measurable overhead to Zed query times, we can revisit this and
hopefully mapstructure will be more mature.  For now, I wanted to
add order.Which to proc.Sort and similar but mapstructure cannot
handle custom JSON unmarshalers so this caused unpack to fail
for apparently no reason.

See mitchellh/mapstructure#115
  • Loading branch information
Brim Automation committed May 11, 2021
1 parent 6ef4b73 commit 74e2b95
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 @@ -155,7 +155,7 @@
"styled-components": "^5.1.1",
"tree-model": "^1.0.7",
"valid-url": "^1.0.9",
"zed": "git+https://github.com/brimdata/zed.git#3dcb327e99cb2d9564af5f316fa07517fa47b5a4"
"zed": "git+https://github.com/brimdata/zed.git#730d19c91f99cb3747143a56c73b5c16b8ba0da6"
},
"optionalDependencies": {
"electron-installer-debian": "^3.0.0",
Expand Down

0 comments on commit 74e2b95

Please sign in to comment.