handler: add option to use new event loop #48138
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This adds support for the new event loop in the handler, controlled by a config option (
new_event_loop=true
) to make it easy to test it without fully committing to it yet.I've done some basic testing of common flows (start/stop, accept sessions, publish messages) and things look ok so far. The main thing missing is support for endpoints that use
SimpleHttpServer
(used by the internal control port and prometheus), as it depends onQTcpSocket
which we don't have a substitute for yet. So the config options for those endpoints (push_in_http_port
andprometheus_port
) need to be omitted when testing, and publishing needs to go over zmq.