mcvsubbu commented on pull request #5685: URL: https://github.com/apache/incubator-pinot/pull/5685#issuecomment-657694285
Also, we don't allow admin to specify different broker tenant names for realtime and offline tables. I would combine the two in the output as well. Keeps it consistent with the table config settings. Otherwise, you run the risk of an admin trying to add broker tenant as "myTenant_REALTIME" or "myTenant_OFFLINE" in the table config. ---------------------------------------------------------------- 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
