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

Engine: implement cb_exit callback for plugins #29

Merged
merged 2 commits into from
Sep 2, 2015

Conversation

pandax381
Copy link
Contributor

Proposal to fix issue #23

Signed-off-by: Masaya YAMAMOTO <pandax381@gmail.com>

Signed-off-by: Masaya YAMAMOTO <pandax381@gmail.com>
@edsiper
Copy link
Member

edsiper commented Sep 1, 2015

Looks good to me, just a minor suggestion, can we call it "cb_exit" instead of cb_pre_exit ?, the exit point is just one.

@pandax381 pandax381 changed the title Engine: implement pre_exit callback for plugins Engine: implement cb_exit callback for plugins Sep 2, 2015
@pandax381
Copy link
Contributor Author

Sure! We call it "cb_exit".

@cosmo0920
Copy link
Contributor

cb_exit is also looks good to me. 👍

edsiper added a commit that referenced this pull request Sep 2, 2015
Engine: implement cb_exit callback for plugins
@edsiper edsiper merged commit 7719138 into fluent:master Sep 2, 2015
@edsiper
Copy link
Member

edsiper commented Sep 2, 2015

thanks!

fujimotos pushed a commit to fujimotos/fluent-bit that referenced this pull request Jul 22, 2019
in_dummy: add new documentation about in_dummy
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