Skip to content

Add extractor predicates #2984

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

Merged
merged 10 commits into from
Mar 2, 2023
Merged

Add extractor predicates #2984

merged 10 commits into from
Mar 2, 2023

Conversation

jachris
Copy link
Contributor

@jachris jachris commented Mar 2, 2023

Extractors such as id.orig_h and :uint64 are now allowed where a predicate is expected. They expand to id.orig_h != nil and :uint64 != nil, respectively.

Closes tenzir/issues#50.

@jachris jachris added the feature New functionality label Mar 2, 2023
@jachris jachris force-pushed the topic/extractor-predicates branch 2 times, most recently from 6b3e1ec to 35ecbf9 Compare March 2, 2023 09:19
@jachris jachris marked this pull request as ready for review March 2, 2023 09:23
@dominiklohmann dominiklohmann requested a review from tobim March 2, 2023 10:03
@jachris jachris requested a review from tobim March 2, 2023 13:02
Copy link
Member

@tobim tobim left a comment

Choose a reason for hiding this comment

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

Thank you for adding this nice opportunity for query golfing.

@jachris jachris force-pushed the topic/extractor-predicates branch from 5dda47c to 1b1c591 Compare March 2, 2023 15:47
@jachris jachris enabled auto-merge March 2, 2023 15:51
@jachris jachris merged commit eb9df84 into master Mar 2, 2023
@jachris jachris deleted the topic/extractor-predicates branch March 2, 2023 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants