You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2022/02/10 21:18:26 UTC

[GitHub] [pinot] Jackie-Jiang commented on issue #8187: API for Fetching Live Brokers of a Table Doesn't Support Table Names without Type Suffix

Jackie-Jiang commented on issue #8187:
URL: https://github.com/apache/pinot/issues/8187#issuecomment-1035528134


   Let's change the existing API to take table name (with or without type suffix).
   - If type suffix is provided, we follow the existing behavior
   - If type suffix is not provided, check the table existence. If only one of OFFLINE/REALTIME table exists, follow the existing behavior; if both table exists, return the intersection of the hosts
   
   @chenboat Do you agree on the proposed change? I remember suggesting this behavior in the original PR, but we decided to do it later?


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org