Repository navigation
docs: pipeline: outputs: opensearch: gate drop_unrecoverable_records to v5.1.3 - #2735
Conversation
…to v5.1.3
drop_unrecoverable_records is not present in v5.1.2 or any earlier tag.
Upstream commit 54f00f219 added it along with the bulk failure summary
log, and both ship in v5.1.3.
- Annotate the configuration parameters row.
- Scope the Bulk retries paragraph that introduces the option.
- Scope the failure summary paragraph, and split its first sentence to
stay under the 35 word vale limit.
The per-document payload rebuild described earlier in that section is
left ungated. retry_payload and next_retry_payload are both present in
v5.1.2, so that filtering, and its trace record carve-out, are accurate
for current releases.
Refs fluent#2727
Signed-off-by: Eric D. Schabell <eric@schabell.org>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review. 📝 WalkthroughWalkthroughThe OpenSearch output documentation now specifies Fluent Bit 5.1.3 and later for ChangesOpenSearch output version documentation
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Other Merge Risk: ⚪ Minimal · up to No concrete merge-blocking issue was established. The documentation change is consistent with the available implementation history. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 1 system. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
|
@patrick-stephens ready for review! |
drop_unrecoverable_records is not present in v5.1.2 or any earlier tag.
Upstream commit 54f00f219 added it along with the bulk failure summary
log, and both ship in v5.1.3.
stay under the 35 word vale limit.
The per-document payload rebuild described earlier in that section is
left ungated. retry_payload and next_retry_payload are both present in
v5.1.2, so that filtering, and its trace record carve-out, are accurate
for current releases.
Refs #2727
Summary by CodeRabbit
drop_unrecoverable_recordsand bulk-failure summaries are available in Fluent Bit 5.1.3 and later.