|
| 1 | + |
| 2 | +TODO |
| 3 | +---- |
| 4 | + |
| 5 | + |
| 6 | +Finish in a scope of the current ticket DM-42005 before the X-Mas break: |
| 7 | + |
| 8 | +- [**x**] Think about the locking mechanism of the method WorkerHttpRequest::toJson(). The method |
| 9 | + acquires a lock on the mutext while the request may too have a lock on the same mutex |
| 10 | + while processing the request in WorkerHttpRequest::execute(). This may result in a deadlock. |
| 11 | + Perhaps no locking is needed as all since the resulting data are not lock sencitive? |
| 12 | +- [**x**] Finish implementing a hierachy of the HTTP-based worker requests |
| 13 | +- [**x**] Finish implementing the request processor for these requests |
| 14 | +- [**x**] Add the new service to the Condfiguration and Registry to allow the Controller to send requests |
| 15 | + to the worker via HTTP |
| 16 | +- [**x**] Display connection parameters of the new service on the Web Dashboard |
| 17 | +- [ ] Document the REST services in the documentation tree. |
| 18 | +- [ ] Manually test the new implementation externally using ``curl`` or Python's ``requests`` module. |
| 19 | + Think about the test cases to cover the new implementation. |
| 20 | +- [ ] Extend the integration tests to cover the new implementation. |
| 21 | + |
| 22 | +Finish in a scope of a separate ticket during/after the X-Mas break: |
| 23 | + |
| 24 | +- [ ] Implement the MessengerHttp on the Controller side of the protocol. The class will |
| 25 | + be providing the multiplexing API for the Controller to send requests to the worker. |
| 26 | + The initial implementation will be based on the simple http::AsyncReq. |
| 27 | +- [ ] Create a parallel hierarchy of the HTTP-based request & job classes on the Controller |
| 28 | + side of the protocol. |
| 29 | +- [ ] Test the new classes. |
| 30 | +- [ ] Implement the MessengerHttp to reuse the socket connections for sending multiple requests |
| 31 | + to the same worker. |
| 32 | +- [ ] Test the new implementation to ensure it works the same way as the old one. |
| 33 | +- [ ] Remove the old implementation of the Controller - Worker protocol. |
| 34 | + |
0 commit comments