Skip to content

Conversation

@marcosdiez
Copy link
Contributor

@marcosdiez marcosdiez commented May 12, 2022

Documentation for fluent/fluent-bit#5449

@lecaros lecaros requested review from a team as code owners March 25, 2025 14:34
@eschabell eschabell requested a review from esmerel October 22, 2025 19:48
@eschabell eschabell self-assigned this Oct 22, 2025
@eschabell eschabell added conflict Waiting on conflict to be resolved by contributor waiting-on-review Waiting on a review from mainteners labels Oct 22, 2025
@eschabell
Copy link
Collaborator

@esmerel can you please review this PR, as I've nudged the owner of the corresponding code PR fluent/fluent-bit#5449 to move that forward too.

@esmerel esmerel removed the conflict Waiting on conflict to be resolved by contributor label Oct 27, 2025
Copy link
Contributor

@esmerel esmerel left a comment

Choose a reason for hiding this comment

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

This adds a bunch of options, I've updated the table per our style and fixed the merge conflict

Signed-off-by: Lynette  Miles <[email protected]>
@esmerel esmerel removed the waiting-on-review Waiting on a review from mainteners label Oct 27, 2025
Copy link
Contributor

@patrick-stephens patrick-stephens left a comment

Choose a reason for hiding this comment

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

I think there are some fundamental implementation concerns to resolve but we seem to have removed the logical_op key in the docs as well.

| `Regex` | `KEY REGEX` | String | Keep records in which the content of `KEY` matches the regular expression. |
| `Exclude` | `KEY REGEX` | String | Exclude records in which the content of `KEY` matches the regular expression. |
| `number_equal` | `KEY NUMBER` | number | Keep records in which the content of `KEY` is equal to `NUMBER`. |
| `number_not_equal` | `KEY NUMBER` | number | Keep records in which the content of `KEY` is not equal to `NUMBER`. |
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not a fan of this, why have we removed the logical_op as well?

I would expect us to add additional operators really rather than weird looking text strings.

It needs examples, e.g. to show precedence and conflict resolution when you define multiple rules for example.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants