cbalci opened a new pull request #7037:
URL: https://github.com/apache/incubator-pinot/pull/7037
## Description
Switch to using 'authority' instead of 'host' when building parent URI in
PinotFS:move method.
This fixes a bug when a FileSystem implementation uses URI with ports. Using
'host' strips the port information from the URI as:
```
URI uri = new URI("hdfs://localhost:1234/my/path");
uri.getHost(); // "localhost"
uri.getAuthority(); // "localhost:1234"
```
- Also added a test case which captures the failure.
## Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order:
Controller, Broker, Server, Minion)
* [ ] Yes (Please label as **<code>backward-incompat</code>**, and complete
the section below on Release Notes)
Does this PR fix a zero-downtime upgrade introduced earlier?
* [ ] Yes (Please label this as **<code>backward-incompat</code>**, and
complete the section below on Release Notes)
Does this PR otherwise need attention when creating release notes? Things to
consider:
- New configuration options
- Deprecation of configurations
- Signature changes to public methods/interfaces
- New plugins added or old plugins removed
* [ ] Yes (Please label this PR as **<code>release-notes</code>** and
complete the section on Release Notes)
--
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
