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

Blocking queue (from @cypof, with modifications) #2167

Closed
wants to merge 3 commits into from

Conversation

longjon
Copy link
Contributor

@longjon longjon commented Mar 20, 2015

This is the blocking queue taken from the parallel branch, modified to remove some features and add others. (By the way, I looked for a more "standard" implementation of this, e.g., in boost, but did not find one.)

I'll splitting this out from a future PR for easier review. There's no need to consider it until that PR is also agreed upon, since this serves no other purpose yet.

@cypof, as with #2067, let me know that the author information is as you prefer.

@longjon longjon force-pushed the blocking-queue branch 2 times, most recently from cb80038 to 72dabac Compare March 20, 2015 06:37
@cypof
Copy link
Member

cypof commented Mar 21, 2015

@longjon author info is fine, thanks. I will fix my config for future commits.

@sguada
Copy link
Contributor

sguada commented Mar 21, 2015

@longjon you may want to add a max_size and block push when the queue is full.

@shelhamer
Copy link
Member

Closing as replaced by #4563

@shelhamer shelhamer closed this Apr 13, 2017
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.

4 participants