xiangfu0 commented on code in PR #14654:
URL: https://github.com/apache/pinot/pull/14654#discussion_r1883145999
##########
pinot-spi/src/main/java/org/apache/pinot/spi/stream/StreamMetadataProvider.java:
##########
@@ -106,6 +107,33 @@ default Map<String, PartitionLagState>
getCurrentPartitionLagState(
return result;
}
+ /**
+ * Fetches the list of available topics/streams
+ *
+ * @param timeout Timeout for fetching the list of topics. If this is null,
the implementation should use a default
+ * timeout.
+ * @return List of topics
+ */
+ default List<TopicMetadata> listTopics(Duration timeout) {
+ throw new UnsupportedOperationException();
+ }
+
+ /**
+ * Represents the metadata of a topic. This can be used to represent the
topic name and other metadata in the future.
+ */
+ class TopicMetadata {
Review Comment:
Or make TopicMetadata an interface then you can have KafkaTopicMetadata
--
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]
