You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
the worker pool in ants looks ok-ish to user but it has some deficiencies:
each job is simply a function func() and there are 2 problems with this:
no function result; what happens if we wish to submit a job that needs a result, eg a calculation
no error returned
propensity to use interface{}; not type safe; prefer to use generics instead
no top level context; so how do we cancel?
submitting jobs requires invoking Pool.Submit, with this, we lose out on the benefit of loose coupling. Now the client must have access to the pool, rather than sending jobs into a channel (our wrapper can fix this)
no concept of a sequence no for each job executed; I'll add this as a feature
ants uses its own logger, when it should just use the one provided by the client
The source code will be copied into lorax and heavily modified, but its core will remain the same. I have no need for a multi pool, so this wil not be used.
This task will port the stretchr tests to use ginko (but actually as we already know the worker-pool works, do not implement all the tests straight away, just do a couple of key tests. Only once we have established that we can make our own changes to the core, will we implment all the tests; I don't want to repeat the mistake I made with rxgo, where I implememnted all the units tests ajust to find out at the end, that rxgo was unsuitable).
The idea behind the use of ants will be to retain most of the user interface we already defined in the legacy work pool but use the ants worker pool under the covers. One change we'll make will be to allow the job to specific an executive function that will override the one defined on the pool; this will allow many different types of execution to occur in the ppol, not just the default one defined on the pool.
The text was updated successfully, but these errors were encountered:
the worker pool in ants looks ok-ish to user but it has some deficiencies:
The source code will be copied into lorax and heavily modified, but its core will remain the same. I have no need for a multi pool, so this wil not be used.
This task will port the stretchr tests to use ginko (but actually as we already know the worker-pool works, do not implement all the tests straight away, just do a couple of key tests. Only once we have established that we can make our own changes to the core, will we implment all the tests; I don't want to repeat the mistake I made with rxgo, where I implememnted all the units tests ajust to find out at the end, that rxgo was unsuitable).
The idea behind the use of ants will be to retain most of the user interface we already defined in the legacy work pool but use the ants worker pool under the covers. One change we'll make will be to allow the job to specific an executive function that will override the one defined on the pool; this will allow many different types of execution to occur in the ppol, not just the default one defined on the pool.
The text was updated successfully, but these errors were encountered: