chenboat commented on pull request #5314: URL: https://github.com/apache/incubator-pinot/pull/5314#issuecomment-622578544
> You can just change the arguments of SplitSegmentCommitter constructor to include TableConfig instea of indexLoadingConfig and commit this PR without actually handling the null case. After we finalize the config, we can handle the null case. > > Meanwhile, we can proceed with other PRs (e.g. a peer downloader) towards finishing the feature I will file the peer download PR soon. We can do these two PRs in parallel. @mcvsubbu ---------------------------------------------------------------- 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
