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

fix producer callback function #109

Merged
merged 2 commits into from
Nov 10, 2020
Merged

fix producer callback function #109

merged 2 commits into from
Nov 10, 2020

Conversation

xuxiaoahang2018
Copy link
Collaborator

  1. 修复producer 回调函数打印为空的问题。

该问题会由以下两种情况下出现:
(1). 程序启动就错误,发送数据量非常少。因为producer异步发送,在消息未被发送时,已经接收到了retry关闭信号,这时候没有去给producerBatch 添加attemp 信息。
(2). 遇到特定的错误码,没有给producerBatch 添加attemp 信息,例如 ProjectNotExist等 .

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


wb-xxh498912 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@shabicheng
Copy link
Collaborator

/LGTM

@shabicheng shabicheng merged commit 9c99819 into master Nov 10, 2020
crimson-gao pushed a commit that referenced this pull request Jan 24, 2024
* Revert GetLogs

* sync code

* sync code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants