[GitHub] [incubator-pinot] mcvsubbu commented on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion

mcvsubbu commented on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion URL: https://github.com/apache/incubator-pinot/pull/4914#issuecomment-610097015 @chenboat I believe @Jackie-Jiang suggested we don't go with another class, but with a table config

[GitHub] [incubator-pinot] mcvsubbu commented on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion

mcvsubbu commented on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion URL: https://github.com/apache/incubator-pinot/pull/4914#issuecomment-610074062 The config change in step 5 is because we want the segment split-committer to look like this: `

[GitHub] [incubator-pinot] mcvsubbu commented on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion

mcvsubbu commented on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion URL: https://github.com/apache/incubator-pinot/pull/4914#issuecomment-609453001 Hi @jamesyfshao , we need to get this in without blocking you, so I suggest we proceed with small

[GitHub] [incubator-pinot] mcvsubbu commented on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion

mcvsubbu commented on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion URL: https://github.com/apache/incubator-pinot/pull/4914#issuecomment-585384008 Is this POC going to be merged, or are we going to split things as per the design doc?