ankitsultana opened a new pull request #8188:
URL: https://github.com/apache/pinot/pull/8188
## Description
At present the /tables/{tableNameWithType}/livebrokers API doesn't support
table names without the type suffix. Upstream callers that are not aware of the
available types for a given table will have to try the API for both the
table-types to determine the available brokers.
We can either update the existing API to handle tables names without type or
add a new API. Updating the existing API would inevitably change the behavior
of the API so it can cause compatibility issues.
This was also discussed in the PR for adding the livebrokers API as a
follow-up:
https://github.com/apache/pinot/commit/e2e3aebab2712accc15190d6d3984b7c9e6de5d7#r738959901
Test Plan: I have added a UT. Also tested on a local Pinot instance.
## Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order:
Controller, Broker, Server, Minion)
* No
Does this PR fix a zero-downtime upgrade introduced earlier?
* No
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)
## Release Notes
<!-- If you have tagged this as either backward-incompat or release-notes,
you MUST add text here that you would like to see appear in release notes of
the
next release. -->
## Documentation
<!-- If you have introduced a new feature or configuration, please add it to
the documentation as well.
See
https://docs.pinot.apache.org/developers/developers-and-contributors/update-document
-->
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
