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.

Sign up for GitHub

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

Jump to bottom

Kafka Logger. #1312

Merged
merged 4 commits into from
Mar 25, 2020
Merged

Kafka Logger. #1312

merged 4 commits into from
Mar 25, 2020

Conversation

Akayeshmantha
Copy link
Member

Adding kafka plugin.

Issues resolved
Fix #1259

@Akayeshmantha
Copy link
Member Author

@moonming @membphis can we review the PR ?

membphis
membphis reviewed Mar 21, 2020
View reviewed changes
.travis.yml Outdated Show resolved Hide resolved
@membphis
Copy link
Member

@Akayeshmantha please rebase your branch with master branch

@Akayeshmantha
Copy link
Member Author

@membphis done

membphis
membphis previously approved these changes Mar 24, 2020
View reviewed changes
@membphis
Copy link
Member

@Akayeshmantha It missing doc now.

The source code is fine.

@Akayeshmantha
Copy link
Member Author

Added @membphis. And rebased with the master 😃

@moonming moonming merged commit 8adafa3 into apache:master Mar 25, 2020
@membphis
Copy link
Member

@Akayeshmantha many thx

SaberMaster pushed a commit to SaberMaster/incubator-apisix that referenced this pull request Jun 30, 2020
@SaberMaster
feature: add Kafka Logger plugin. (apache#1312)
79b2339
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@membphis membphis membphis approved these changes

@moonming moonming moonming approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add support to kafka publisher plugin
3 participants
@Akayeshmantha @membphis @moonming