lidavidm commented on PR #1027: URL: https://github.com/apache/arrow-java/pull/1027#issuecomment-5472093363
> Validation done, all tests pass locally. > > For the validation, I allowed port 0 to be used for compatibility with a few existing tests that used it. I don't think we should allow it though: this is a client code path, it is meaningless to try to connect to port 0. > > If you agree with this assessment @lidavidm I'll go ahead and create a separate issue+PR for removing 0 from the valid range (we can leave it for a next release). Sounds good to me. -- 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]
