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

[READY] [FIXED JENKINS-19691] - Added the hudson.model.Queue.cacheRefreshPeriod option #1221

Conversation

4 participants
@oleg-nenashev
Copy link
Member

commented May 2, 2014

The option allows to manage the refresh period of the internal queue cache.

Signed-off-by: Oleg Nenashev o.v.nenashev@gmail.com

@cloudbees-pull-request-builder

This comment has been minimized.

Copy link

commented May 2, 2014

core » jenkins-core #617 UNSTABLE
Looks like there's a problem with this pull request

@oleg-nenashev oleg-nenashev changed the title [FIXED JENKINS-19691] - Added the hudson.model.Queue.cacheRefreshPeriod option [READY ][FIXED JENKINS-19691] - Added the hudson.model.Queue.cacheRefreshPeriod option May 21, 2014

@oleg-nenashev oleg-nenashev changed the title [READY ][FIXED JENKINS-19691] - Added the hudson.model.Queue.cacheRefreshPeriod option [READY] [FIXED JENKINS-19691] - Added the hudson.model.Queue.cacheRefreshPeriod option May 21, 2014

@oleg-nenashev

This comment has been minimized.

Copy link
Member Author

commented Jun 22, 2014

Any feedback?

@daniel-beck

This comment has been minimized.

Copy link
Member

commented Jul 12, 2014

Is this still a problem since the unnecessary disk accesses were eliminated (JENKINS-22822)?

@oleg-nenashev

This comment has been minimized.

Copy link
Member Author

commented Aug 14, 2014

@daniel-beck
Yes, the issue appears with this fix as well.
In my case it may be caused by the Throttling overhead.

@olivergondza

This comment has been minimized.

Copy link
Member

commented Aug 14, 2014

@oleg-nenashev Can we un-final the field so it can be tweaked from groovy console easily?

@oleg-nenashev

This comment has been minimized.

Copy link
Member Author

commented Aug 14, 2014

Agreed

[FIXED JENKINS-19691] - Added the hudson.model.Queue.cacheRefreshPeri…
…od option

The option allows to manage the refresh period of the internal queue cache.

Signed-off-by: Oleg Nenashev <o.v.nenashev@gmail.com>
@cloudbees-pull-request-builder

This comment has been minimized.

Copy link

commented Aug 14, 2014

core » jenkins-core #1279 SUCCESS
This pull request looks good

@olivergondza olivergondza merged commit 0d82598 into jenkinsci:master Aug 14, 2014

olivergondza added a commit that referenced this pull request Aug 14, 2014

Merge pull request #1221 from synopsys-arc-oss/configurable_queue_cac…
…he_refresh

[READY] [FIXED JENKINS-19691] - Added the hudson.model.Queue.cacheRefreshPeriod option

olivergondza added a commit that referenced this pull request Aug 14, 2014

@olivergondza

This comment has been minimized.

Copy link
Member

commented Aug 14, 2014

Merged, thanks!

@daniel-beck

This comment has been minimized.

Copy link
Member

commented Oct 19, 2014

@oleg-nenashev It's time you documented this here.

@oleg-nenashev

This comment has been minimized.

Copy link
Member Author

commented Oct 19, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.