Benjamin Zikarsky
Benjamin Zikarsky
Fix this and add proper tests.
Gearman added several new commands at http://gearman.org/protocol/: ``` 35 SUBMIT_JOB_SCHED REQ Client 36 SUBMIT_JOB_EPOCH REQ Client 37 SUBMIT_REDUCE_JOB REQ Client 38 SUBMIT_REDUCE_JOB_BACKGROUND REQ Client 39 GRAB_JOB_ALL REQ Worker 40 JOB_ASSIGN_ALL...
Relates to #10: react-gearman relies on react/socket-client 0.4.* which got upgraded to 0.7, depreacted and replaced by react/socket. All react/socket-client usages have to be upgraded to react/socket.
In regards to #10: Currently react/stream is fixed to 0.4.*. The most recent version is 0.7. We cannot upgrade due to a number of BC breaks between 0.4 and 0.7....
Currently there are a lot of composer dependencies (especiallyreact/*) which are below 1.0 but defined with `~`. Semver allows breaking changes between 0.x and 0.x+1. Therefore this is not safe....
See https://github.com/bzikarsky/gearman-async/blob/95e9b5e4a877ca102501ee7449038282b5914781/src/Gearman/Async/Client.php#L100
- Add a class `ClientPool` (implementing `ClientInterface`); additional methods - `addClient($hostname, $port)` - `setDistributionStrategy($strategy)` - `getDistributionStrategy()` - Add a factory method `createClientPool($strategy = null)` The client-pool should chose a client...
Currently the documentation is almost non existent. This should be changed. - [ ] Explain architecture with comprehensive examples - [ ] Explain integration into monolog - [ ] Update...
Currently there are 2 tests: For TCP and UDP usage. TCP tests all the general functionality and TCP specifics; UDP tests only some UDP stuff. This needs to be refactored...