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

modify static fields/methods to non-static for less memory consumption #4

Merged

Conversation

@Shwetajain148
Copy link

@Shwetajain148 Shwetajain148 commented Feb 28, 2017

@mchaudhary, Now I can see library is consuming less memory.

Currently, I am seeing some indentation related issues in my PR, that I will correct and raise the PR again.

@Shwetajain148 Shwetajain148 changed the title modify static fields/methods to non-static and enhanced memory consumption modify static fields/methods to non-static for less memory consumption Feb 28, 2017
@Shwetajain148 Shwetajain148 force-pushed the Shwetajain148:memory-performance-improvement branch from a1d954f to 84707f7 Mar 2, 2017
@Shwetajain148 Shwetajain148 force-pushed the Shwetajain148:memory-performance-improvement branch from 84707f7 to 8c1643a Mar 2, 2017
@Shwetajain148
Copy link
Author

@Shwetajain148 Shwetajain148 commented Mar 2, 2017

@mchaudhary, @mostlyjason I have corrected indentation for this PR. Please review.

@mchaudhary mchaudhary merged commit 6e0cbac into loggly:master Mar 9, 2017
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

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