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

decrease waiting time to send bulk logs #13

Conversation

@Shwetajain148
Copy link

@Shwetajain148 Shwetajain148 commented Aug 8, 2017

@mchaudhary, @mostlyjason, This PR is to decrease the waiting time to send bulk logs. Before library waits for 30 seconds to collect logs and then sends them to Loggly but now library will send Logs in each 5 seconds.

@Shwetajain148
Copy link
Author

@Shwetajain148 Shwetajain148 commented Sep 13, 2017

@mchaudhary @mostlyjason, This PR's code has already been merged so closing this one.

@Shwetajain148 Shwetajain148 deleted the Shwetajain148:change-library-time-to-send-bulk-logs branch Sep 13, 2017
@dncrews
Copy link

@dncrews dncrews commented Nov 16, 2017

Is there a chance of getting a release of this? Your library is now being used by bunyan-loggly, and the 30 second timeout is breaking all AWS Lambda usage.

@mostlyjason
Copy link

@mostlyjason mostlyjason commented Nov 16, 2017

I believe the latest release has the timeout set to five seconds https://github.com/loggly/node-loggly-bulk/blob/master/lib/loggly/common.js#L240. Are you on version 2.2.0?

@dncrews
Copy link

@dncrews dncrews commented Nov 16, 2017

Nope. That's my bad. I only looked at the releases in GitHub, not the actual package. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.