somandal merged PR #15637:
URL: https://github.com/apache/pinot/pull/15637
--
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.apa
codecov-commenter commented on PR #15637:
URL: https://github.com/apache/pinot/pull/15637#issuecomment-2829002859
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15637?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
somandal commented on code in PR #15637:
URL: https://github.com/apache/pinot/pull/15637#discussion_r2059291365
##
pinot-controller/src/main/java/org/apache/pinot/controller/util/TableMetadataReader.java:
##
@@ -71,25 +72,26 @@ public TableReloadJsonResponse
getServerCheckSegme
Jackie-Jiang commented on code in PR #15637:
URL: https://github.com/apache/pinot/pull/15637#discussion_r2059255083
##
pinot-controller/src/main/java/org/apache/pinot/controller/util/TableMetadataReader.java:
##
@@ -71,25 +72,26 @@ public TableReloadJsonResponse
getServerCheckS
somandal opened a new pull request, #15637:
URL: https://github.com/apache/pinot/pull/15637
Fix needReload to fetch status from servers in ExternalView
Today the `needReload` API fetches the status from the servers with the
instance tag corresponding to the Tenant. This is not a true
