[GitHub] [incubator-pinot] kishoreg commented on pull request #5718: Feature/#5390 segment indexing reload status api

2020-08-13 Thread GitBox
kishoreg commented on pull request #5718: URL: https://github.com/apache/incubator-pinot/pull/5718#issuecomment-673284226 > A lot of code duplication with the existing table size reader. Please find ways to use a common base class if possible Is this addressed?

[GitHub] [incubator-pinot] kishoreg commented on pull request #5718: Feature/#5390 segment indexing reload status api

2020-08-09 Thread GitBox
kishoreg commented on pull request #5718: URL: https://github.com/apache/incubator-pinot/pull/5718#issuecomment-671062149 is this ready to go? This is an automated message from the Apache Git Service. To respond to the