Parser plugin docs updates#2371
Conversation
- Sort parameter table alphabetically - Add Default column with values for each parameter - Add missing logfmt_no_bare_keys parameter - Add missing mysql_quoted decoder type to decode_field and decode_field_as - Fix Time_System_timezone case to time_system_timezone - Lowercase all parameter names to match repo YAML convention - Remove inaccurate format restriction on types parameter Applies to fluent#2370 Signed-off-by: Eric D. Schabell <eric@schabell.org>
- Restructure sections to Configuration parameters before Examples - Sort decoder options table alphabetically - Sort optional actions table alphabetically - Add missing mysql_quoted decoder type - Lowercase parameter references to match repo YAML convention Applies to fluent#2370 Signed-off-by: Eric D. Schabell <eric@schabell.org>
Applies to fluent#2370 Signed-off-by: Eric D. Schabell <eric@schabell.org>
- Add link to configuring-parser for common parameters - Add format-specific parameter table for logfmt_no_bare_keys Applies to fluent#2370 Signed-off-by: Eric D. Schabell <eric@schabell.org>
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📝 WalkthroughWalkthroughStandardizes parser documentation keys from CamelCase to snake_case across multiple parser docs, adds configuration parameter tables and cross-references to the main configuring-parsers guide, and documents new parser options (e.g., mysql_quoted, logfmt_no_bare_keys, key_group, key_pattern). All edits are documentation-only. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@pipeline/parsers/ltsv.md`:
- Line 1: Remove the leading spaces before the Markdown header so the header
token starts at column 0; locate the header line that contains "# LTSV format"
and delete any preceding whitespace so it becomes exactly "# LTSV format" with
no indentation.
🧹 Nitpick comments (1)
pipeline/parsers/multiline-parsing.md (1)
44-44: Consider rewording to address style guideline.The description uses "useful," which the style guide discourages as it implies ease of use. Consider a more direct phrasing.
Alternative wording
-| `key_pattern` | For an incoming structured message, specify an alternative key to apply matching rules against, separate from `key_content`. This is useful when you want to match against one field while concatenating content from another. | _none_ | +| `key_pattern` | For an incoming structured message, specify an alternative key to apply matching rules against, separate from `key_content`. This allows matching against one field while concatenating content from another. | _none_ |
74f3e6c to
5041702
Compare
- Add link to configuring-parser for common parameters
- Lowercase skip_empty_values and its default to match repo convention
- Fix {% end hint %} to {% endhint %}
Applies to fluent#2370
Signed-off-by: Eric D. Schabell <eric@schabell.org>
- Add key_group parameter for stream grouping - Add key_pattern parameter for alternative match field Applies to fluent#2370 Signed-off-by: Eric D. Schabell <eric@schabell.org>
Applies to fluent#2370 Signed-off-by: Eric D. Schabell <eric@schabell.org>
5041702 to
a4fdee4
Compare
|
@cosmo0920 or @patrick-stephens, ready for review! |
Fixes #2370
Summary by CodeRabbit