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

How to configure TimeoutPersisterReceiver.SecondsToSleepBetweenPolls #2316

Closed
SimonCropp opened this issue Aug 18, 2014 · 9 comments
Closed

Comments

@SimonCropp
Copy link
Contributor

so according to this http://docs.particular.net/nservicebus/configure_timeout_polling_interval you configure SecondsToSleepBetweenPolls by setting it directly on TimeoutPersisterReceiver inside a IWantToRunWhenBusStartsAndStops.

This is problematic since TimeoutPersisterReceiver is now internal.

So do we add an explicit config API for SecondsToSleepBetweenPolls?

@SimonCropp SimonCropp added this to the 5.0.0 milestone Aug 18, 2014
@johnsimons
Copy link
Member

When did that become a configurable setting?

On 18 August 2014 10:21, Simon Cropp notifications@github.com wrote:

so according to this
http://docs.particular.net/nservicebus/configure_timeout_polling_interval
you configure SecondsToSleepBetweenPolls by setting it directly on
TimeoutPersisterReceiver inside a IWantToRunWhenBusStartsAndStops.

This is problematic since TimeoutPersisterReceiver is now internal.

So do we add an explicit config API for SecondsToSleepBetweenPolls?


Reply to this email directly or view it on GitHub
#2316.

@SimonCropp
Copy link
Contributor Author

@johnsimons since it is a public property on a public type. if it is not meant to be changed by the user feel free to delete the doco page

@SimonCropp
Copy link
Contributor Author

as an aside I think this is used by people on azure to reduce the cost of polling too often

@SeanFeldman
Copy link
Contributor

I might be mistaken, but Azure Storage queue transport defines its own mechanism.
//cc @yvesgoeleven

@andreasohlund
Copy link
Member

How about an appsetting?

Sent from my iPhone

On 18 Aug 2014, at 02:21, Simon Cropp notifications@github.com wrote:

so according to this http://docs.particular.net/nservicebus/configure_timeout_polling_interval you configure SecondsToSleepBetweenPolls by setting it directly on TimeoutPersisterReceiver inside a IWantToRunWhenBusStartsAndStops.

This is problematic since TimeoutPersisterReceiver is now internal.

So do we add an explicit config API for SecondsToSleepBetweenPolls?


Reply to this email directly or view it on GitHub.

@SimonCropp SimonCropp modified the milestones: 5.0.0, 5.1.0 Aug 18, 2014
@SimonCropp
Copy link
Contributor Author

-1 for an appsetting

if this is not a publicly used api, as john suspects, then it can be deferred. so moving to 5.1

@andreasohlund
Copy link
Member

Ok, let's defer

On Mon, Aug 18, 2014 at 11:56 AM, Simon Cropp notifications@github.com
wrote:

-1 for an appsetting

if this is not a publicly used api, as john suspects, then it can be
deferred. so moving to 5.1


Reply to this email directly or view it on GitHub
#2316 (comment)
.

@johnsimons
Copy link
Member

Can we close this?
I have never see a requirement to change the default, has anyone?

@andreasohlund
Copy link
Member

Yea, lets close. We'll reopen if we starting to get user requests for this

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

No branches or pull requests

4 participants