Skip to content
This repository was archived by the owner on Nov 14, 2023. It is now read-only.

update jsonpath library to support async compute - #6

Merged
Eoin Power-Moran (e-moran) merged 9 commits into
masterfrom
eoinpm/etr-1303-update-jsonpath-library-to-support-async-compute
Oct 19, 2023
Merged

update jsonpath library to support async compute#6
Eoin Power-Moran (e-moran) merged 9 commits into
masterfrom
eoinpm/etr-1303-update-jsonpath-library-to-support-async-compute

Conversation

@e-moran

Copy link
Copy Markdown

Why

Need to be able to compute the value for replace_with asynchronously using multiple selectors so we can send it to an external service for processing.

How

Add a new type of selector that processes multiple jsonpaths at once.
Add a new path struct to hold metadata and associated it with found fields.

@e-moranEoin Power-Moran (e-moran) changed the title Eoinpm/etr 1303 update jsonpath library to support async computeupdate jsonpath library to support async computeOct 19, 2023
Comment threadbenches/common.rs Outdated
)
})
.parse_env("RUST_LOG")
// .filter(Some("logger_example"), LevelFilter::Trace)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this line commented on purpose?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really neat!

Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@e-moran@jeremywagemans