wp-1 removed defer

[ghost]

No description provided.

[codecov-io]

Codecov Report

Merging #1 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master     #1   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines          32     30    -2     
=====================================
- Hits           32     30    -2

Impacted Files	Coverage Δ
pool.go	100% <100%> (ø)	⬆️

---

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 a6bce63...cb2aca0. Read the comment docs.

[vardius]

Can you explain what problem this solves, how does it help ?

The idea of running grouting here was for Delegate function to not block when adding jobs to full queue. Sends to a buffered channel block only when the buffer is full.

[ghost]

In your delegate function, there are chances that the goroutine won't run at all since delgate function has not other commnd to run after the goroutine has started.Delegate function will sometimes end even before the goroutine has completed

[vardius]

@siddhant2408 could you add a test case for it ? That breaks now and will work after your changes ?