yashmayya opened a new pull request, #13236: URL: https://github.com/apache/pinot/pull/13236
- Currently, the CLI options `-realTimeInstanceCount` and `-offlineInstanceCount` are marked as required for the `AddTenant` command. - However, these options are only needed when creating server tenants and not when creating broker tenants (see [here](https://github.com/apache/pinot/blob/518fd180981ffa614042b8a5236650a2af41bfe5/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java#L1312-L1363)). We shouldn't unnecessarily require users to also configure these CLI options when creating broker tenants (also, it's not documented here - https://docs.pinot.apache.org/basics/concepts/components/cluster/tenant#broker-tenant). -- 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-unsubscr...@pinot.apache.org 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
