jenkinsci / scm-sync-configuration-plugin

Jenkins scm-sync-configuration plugin
https://plugins.jenkins.io/scm-sync-configuration/
MIT License
115 stars 123 forks source link

[JENKINS-14893] reduce memory consumption / OOME #10

Closed mhelff closed 11 years ago

mhelff commented 11 years ago

Hi,

this is rather simple: Instead of reading the logs always, the change checks first if the user enabled "displayStatus". Solved the memory consumption issues we had on our Jenkins instance.

Martin

buildhive commented 11 years ago

Jenkins » scm-sync-configuration-plugin #28 SUCCESS This pull request looks good (what's this?)

johnou commented 11 years ago

Thank you for your contribution!

fcamblor commented 11 years ago

thanks :-)

mhelff commented 11 years ago

thank you for the fast merge! looking forward to the next release of the scm-sync-configuration-plugin :-)

johnou commented 11 years ago

I just tried it on on my Windows development environment but there are problems with the unit tests scm:svn:file:// vs scm:svn:file:/// will run the release again on Linux soon.

johnou commented 11 years ago

0.0.7 should appear in your plugin manager soon.

fcamblor commented 11 years ago

'would have been happy to be part of the release process... specifically for the version naming.

would have named it 0.0.6.2 since it is much liker a bugfix version than a feature extension version

johnou commented 11 years ago

Please accept my apologies Frédéric, I will be sure to consult with you in future regarding changes to scm-sync-configuration-plugin.

fcamblor commented 11 years ago

At least, don't forget to update the wiki for this release

johnou commented 11 years ago

Wiki page updated.