kishoreg commented on pull request #5632:
URL: https://github.com/apache/incubator-pinot/pull/5632#issuecomment-651585940
@chenboat, is there a PR associated here. I thought the PR you were working
on was making deep store optional for some cases.
@mcvsubbu, there are two solutions
1. Let the real-time servers push the segments to controllers but the
controller puts the deep store uri in zk segment metadata
2. real-time server commits segment directly to deep store and invoke the
commit API with URI?
I thought this PR was achieving 1 and we can get to 2 in the next iteration.
----------------------------------------------------------------
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
