mcvsubbu commented on issue #6671: URL: https://github.com/apache/incubator-pinot/issues/6671#issuecomment-797162771
There are other issues with the lease extender as well, as pointed out by @Jackie-Jiang . Looks like things changed during controller distribution and the lease extender was not fixed right. Overall, here is what needs to happen: - Lease extended should get instantiated in HelixServerStarter instead of table data manager. It should also get stopped there. It should be instantiated without a table name , of course. - The addSegment API of lease extender should be called with an addition argument of protocol handler , so that the extend lease request goes to the right controller for the table. ---------------------------------------------------------------- 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
