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

feat: add tryEnqueue function. #406

Merged
merged 2 commits into from
Apr 25, 2019
Merged

feat: add tryEnqueue function. #406

merged 2 commits into from
Apr 25, 2019

Conversation

appleboy
Copy link
Owner

tryEnqueue tries to enqueue a job to the given job channel. Returns true if
the operation was successful, and false if enqueuing would not have been
possible without blocking. Job is not enqueued in the latter case.

Signed-off-by: Bo-Yi Wu appleboy.tw@gmail.com

tryEnqueue tries to enqueue a job to the given job channel. Returns true if
the operation was successful, and false if enqueuing would not have been
possible without blocking. Job is not enqueued in the latter case.

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>
Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>
@codecov-io
Copy link

Codecov Report

Merging #406 into master will decrease coverage by 0.07%.
The diff coverage is 75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #406      +/-   ##
==========================================
- Coverage    84.7%   84.63%   -0.08%     
==========================================
  Files          19       19              
  Lines        1314     1321       +7     
==========================================
+ Hits         1113     1118       +5     
- Misses        163      164       +1     
- Partials       38       39       +1
Impacted Files Coverage Δ
gorush/worker.go 96% <75%> (-4%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5b526a7...eea3192. Read the comment docs.

@appleboy appleboy merged commit 27ec425 into master Apr 25, 2019
@appleboy appleboy deleted the enqueue branch April 25, 2019 15:10
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