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

Per-message Requeue #12

Merged
merged 1 commit into from
Oct 4, 2013
Merged

Conversation

davegardnerisme
Copy link
Contributor

Adding Requeue() method to messages for fine-grained control on requeue delays on message-by-message basis.

Copied the same approach used as for Touch().

Copied the same interface for Requeue -- with timeoutMs int rather than a Go Duration - as it seemed a better fit with the existing codebase.

@mreiferson
Copy link
Member

Yea, I agree. I terribly regret not using go time types for the public interface though... sigh

thanks @davegardnerisme

@mreiferson
Copy link
Member

👍

mreiferson added a commit that referenced this pull request Oct 4, 2013
@mreiferson mreiferson merged commit c93df8f into nsqio:master Oct 4, 2013
@davegardnerisme
Copy link
Contributor Author

The main downside is that I always have to go the playground to make sure I'm turning things into milliseconds correctly ;)

http://play.golang.org/p/jAqV6fgHyF

@mreiferson
Copy link
Member

it's even easier...

milliseconds := int(d/time.Millisecond)

@davegardnerisme
Copy link
Contributor Author

Genius. I knew I was being stupid.

On Fri, Oct 4, 2013 at 8:57 PM, Matt Reiferson [email protected]:

it's even easier...

milliseconds := int(d/time.Millisecond)


Reply to this email directly or view it on GitHubhttps://github.com//pull/12#issuecomment-25726081
.

sthulb pushed a commit to HailoOSS/go-nsq that referenced this pull request Sep 14, 2016
Unpin from our fork of go-snappystream
amosir pushed a commit to amosir/go-nsq that referenced this pull request Jan 16, 2024
fix blocking Depth() call on the closed queue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants