chenboat commented on a change in pull request #8188:
URL: https://github.com/apache/pinot/pull/8188#discussion_r813106287
##########
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java
##########
@@ -3181,15 +3181,53 @@ public TableStats getTableStats(String
tableNameWithType) {
}
/**
- * Return the list of live brokers serving the corresponding table.
- * Each entry in the broker list is of the following format:
- * Broker_hostname_port
+ * Return the list of live brokers serving the corresponding table. Based on
the
+ * input tableName, there can be 3 cases:
+ *
+ * 1. If the tableName has a type-suffix, then brokers for only that
table-type
+ * will be returned.
+ * 2. If the tableName doesn't have a type-suffix and there's only 1 type
for that
+ * table, then the brokers for that table-type would be returned.
+ * 3. If the tableName doesn't have a type-suffix and there are both REALTIME
+ * and OFFLINE tables, then the intersection of the brokers for the two
table-types
+ * would be returned. Intersection is taken since the method guarantees
to return
+ * brokers which can serve the given table. In case of no type provided,
it returns
Review comment:
"In case of no type provided..." -- isn't this sentence a repeat of what
is stated right before it?
--
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]
