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

chore(plugins): cherry pick batch queue return value fix to 3.1.x #10126

Merged
merged 2 commits into from
Jan 17, 2023

Conversation

windmgc
Copy link
Member

@windmgc windmgc commented Jan 17, 2023

Summary

Cherry-pick the batch queue return value fix to 3.1.x branch.
PRs that have been cherry-picked:

Checklist

Full changelog

  • [Implement ...]

Issue reference

Fix #[issue number]

#10044)

The log function in Datadog is called by the batch
queue when a batch is processed, and batch queue relys
on the return value of the callback. The return value of log
function in Datadog always return `nil`, this makes batch queue consider
the result of processing as failed. In this commit, the correct return value
indicating the success in processing to fix this bug.
…e callback functions (#10052)

Add return values for functions that will be processed in batch queue for these two plugins.
Batch queue considers the result of processing unsuccessful without a return
value, resulting the batch being reprocessed incorrectly.

FTI-4645
@windmgc windmgc changed the title Cherry pick batch queue return value fix chore(plugins): cherry pick batch queue return value fix to 3.1.x Jan 17, 2023
@hanshuebner hanshuebner merged commit 492596a into release/3.1.x Jan 17, 2023
@hanshuebner hanshuebner deleted the cherry-pick-batch-queue-return-value-fix branch January 17, 2023 11:26
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.

3 participants