lidavidm commented on PR #15031: URL: https://github.com/apache/arrow/pull/15031#issuecomment-1357854486
I looked at adding a unit test - however it would mean refactoring the internals quite a bit to expose the right bits to the unit test. I think that's worth considering separately: - A core class that just implements the state machine for handling calls - A wrapper class that connects the state machine to UCX Then we can test the core state machine more easily. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
