Skip to content
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

Error when the Remap del function attempts to delete a path that doesn't exist #5779

Closed
binarylogic opened this issue Jan 1, 2021 · 1 comment
Assignees
Labels
domain: vrl Anything related to the Vector Remap Language type: enhancement A value-adding code change that enhances its existing functionality.

Comments

@binarylogic
Copy link
Contributor

This is a replacement issue for #5566 to more clearly represent the conclusion there:

So given this, I’m almost inclined to say: when deleting a path, you have to handle the error case for when a path does not exist.

This is blocked on #5479 since we should resolve our error handling primitives first.

@binarylogic binarylogic added type: enhancement A value-adding code change that enhances its existing functionality. domain: vrl Anything related to the Vector Remap Language labels Jan 1, 2021
@binarylogic binarylogic added this to the 2020-12-21 Kryptek Yeti milestone Jan 1, 2021
@JeanMertz
Copy link
Contributor

We opted not to do this, instead documenting the existence of the exists function to make a distinction between existing null fields or non-existing fields. See the discussions in #5633.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain: vrl Anything related to the Vector Remap Language type: enhancement A value-adding code change that enhances its existing functionality.
Projects
None yet
Development

No branches or pull requests

3 participants