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 logstash default bulk max size #1828

Merged
merged 1 commit into from
Jun 10, 2016

Conversation

urso
Copy link

@urso urso commented Jun 9, 2016

Resolves #1662

@ruflin
Copy link
Member

ruflin commented Jun 9, 2016

LGTM.

@urso see second comment in #1662 You also want to add it here?

@urso
Copy link
Author

urso commented Jun 9, 2016

Oh, thanks. Yeah, will introduce it

@ruflin ruflin removed the review label Jun 10, 2016
@ruflin
Copy link
Member

ruflin commented Jun 10, 2016

@urso ok, thanks. I removed the review label so it doesn't get merged by accident.

@urso urso force-pushed the fix/1662-ls-output-default-bulk-size branch from 9dacbaa to c3dce09 Compare June 10, 2016 12:20
@urso urso added the review label Jun 10, 2016
@urso
Copy link
Author

urso commented Jun 10, 2016

@ruflin done

@ruflin ruflin merged commit 8ea266f into elastic:1.3 Jun 10, 2016
@ruflin
Copy link
Member

ruflin commented Jun 10, 2016

@urso not sure if we have to forward port this?

@urso urso deleted the fix/1662-ls-output-default-bulk-size branch February 19, 2019 18:43
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
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.

2 participants