jbonofre opened a new pull request, #1443: URL: https://github.com/apache/arrow-java/pull/1443
Follow-up to #1363. - Don't copy `hostnameOverride` in the `ArrowFlightSqlClientHandler.Builder` copy constructor. The override is only valid for the original host and port, so a copied builder no longer carries it to another location by default. `getStreams` already sets it explicitly for each endpoint, so behavior is unchanged. - Trim `ArrowFlightConnectionConfigImpl.getHostnameOverride()` and return `null` for blank values, as its Javadoc says. - Record the `:authority` the server receives in `ConnectionTlsHostnameOverrideTest`. The plaintext test now checks that the connection's own host and port are sent, and a new test checks that the override replaces the whole authority and that a port can be included. - Document that the override replaces the port in `:authority`, and that a port can be included. Closes #1442. -- 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]
