mcvsubbu commented on issue #5391:
URL: 
https://github.com/apache/incubator-pinot/issues/5391#issuecomment-629520902


   > IMO, servers should not change the segment ZK metadata. It should only be 
managed by the controller. We have multiple replicas of the same segment on 
different servers, and segment can be moved to any server at any time. If we 
allow servers to change the ZK metadata, that will be very hard to manage.
   > I think providing API for querying segment metadata is okay. We can return 
the segment metadata for each replica of a segment.
   
   +1
   
   The data of whether servers loaded inverted index (or any other) should be 
with the servers.
   One way of doing this is to raise a metric on the servers when reload fails, 
and set alert on the metric. In theory, the reload can take infinite time to 
arrive and therefore it is never possible to say when it is "done". In 
practice, however, it should be a short period of time for the message itself 
to be transmitted to the servers, so we should be ok


----------------------------------------------------------------
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

Reply via email to