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

change time interval to 5 second #1

Closed
wants to merge 1 commit into from

Conversation

@Shwetajain148
Copy link

@Shwetajain148 Shwetajain148 commented Nov 16, 2016

In bulk mode, Logs wait for 20000 milliseconds (20 seconds) to send logs to the loggly.
When application exit within 20 second after sending the log, it doesn't reach to the loggly.

We should decrease wait time interval to overcome the above issue.

@Shwetajain148
Copy link
Author

@Shwetajain148 Shwetajain148 commented Nov 23, 2016

@mchaudhary Please review this PR.

@Shwetajain148
Copy link
Author

@Shwetajain148 Shwetajain148 commented Nov 28, 2016

@mchaudhary In this PR, I have added the log4net-loggly.nuspec file that will be helpful to publish new NuGet package in future, and have changed log collection time in bulk mode. Kindly review and merge this PR.

cc: @mostlyjason @psquickitprageet

<package >
<metadata>
<id>log4net-loggly</id>
<version>7.2.2</version>

This comment has been minimized.

@Shwetajain148
Copy link
Author

@Shwetajain148 Shwetajain148 commented Dec 5, 2016

Accidently I had removed my fork's branch. Due to which, this PR was closed.

I have raised a new PR #2.

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.