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

Add example of partial batch failure handling in Destination Function [FUNK] #6922

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from

Conversation

sudojatin
Copy link
Contributor

Add example of partial batch failure for destination function

Proposed changes

This PR adds an example of partial batch failure handling in destination function

Merge timing

ASAP once approved

Related issues (optional)

@sudojatin sudojatin requested review from pwseg and a team as code owners August 9, 2024 09:57
@sudojatin sudojatin requested a review from Ashwithabg August 9, 2024 10:03
@pwseg pwseg changed the title [Funk] Add example of partial batch failure handling in destination function Add example of partial batch failure handling in Destination Function Feb 13, 2025
@pwseg pwseg changed the title Add example of partial batch failure handling in Destination Function Add example of partial batch failure handling in Destination Function [FUNK] Feb 13, 2025
@pwseg pwseg added the content-update updates to content that are not new features, includes grammar fixes, added notes label Feb 13, 2025
pwseg
pwseg previously approved these changes Feb 13, 2025
@sudojatin sudojatin dismissed pwseg’s stale review February 13, 2025 20:39

The merge-base changed after approval.

pwseg
pwseg previously approved these changes Feb 13, 2025
@sudojatin sudojatin dismissed pwseg’s stale review February 22, 2025 05:56

The merge-base changed after approval.

pwseg
pwseg previously approved these changes Mar 6, 2025
@sudojatin sudojatin dismissed pwseg’s stale review March 6, 2025 18:43

The merge-base changed after approval.

pwseg
pwseg previously approved these changes Mar 6, 2025
@pwseg
Copy link
Contributor

pwseg commented Mar 6, 2025

@Ashwithabg Could you take a look at this? Need your review before merging.

@sudojatin sudojatin dismissed pwseg’s stale review March 7, 2025 05:34

The merge-base changed after approval.

@sudojatin sudojatin requested review from twfarhan and removed request for Ashwithabg March 7, 2025 05:34
twfarhan
twfarhan previously approved these changes Mar 7, 2025
Copy link

@twfarhan twfarhan left a comment

Choose a reason for hiding this comment

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

LGTM

@sudojatin sudojatin dismissed twfarhan’s stale review March 7, 2025 09:42

The merge-base changed after approval.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content-update updates to content that are not new features, includes grammar fixes, added notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants