params: move blocking read to Cython and remove C++ signal handling #34407
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.
re-open #32367
The current C++ blocking read have signal handling issues:
params_sig_handler
installed by the preceding blocking read. This action permanently disables the default signal handler or the handler set up by the program, leading to improper signal handling within the program after blocking reads.this PR replaces the problematic C++ blocking reads with Cython implementation. This new implementation delegates the handling of SIGINT and SIGTERM signals to Python's built-in mechanisms, ensuring smoother interruption management.
This PR removes C++ support for blocking reads (as they are no longer needed in the current C++ codebase), though they remain straightforward to implement if required: