[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3979: Track Indexed timestamp across consuming segments

2019-03-18 Thread GitBox
mcvsubbu commented on a change in pull request #3979: Track Indexed timestamp across consuming segments URL: https://github.com/apache/incubator-pinot/pull/3979#discussion_r266569347 ## File path: pinot-core/src/main/java/org/apache/pinot/core/indexsegment/mutable/MutableSegmentImp

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3979: Track Indexed timestamp across consuming segments

2019-03-18 Thread GitBox
mcvsubbu commented on a change in pull request #3979: Track Indexed timestamp across consuming segments URL: https://github.com/apache/incubator-pinot/pull/3979#discussion_r266511438 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/reduce/BrokerReduceService.java

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3979: Track Indexed timestamp across consuming segments

2019-03-18 Thread GitBox
mcvsubbu commented on a change in pull request #3979: Track Indexed timestamp across consuming segments URL: https://github.com/apache/incubator-pinot/pull/3979#discussion_r266514769 ## File path: pinot-core/src/main/java/org/apache/pinot/core/indexsegment/mutable/MutableSegmentImp

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3979: Track Indexed timestamp across consuming segments

2019-03-18 Thread GitBox
mcvsubbu commented on a change in pull request #3979: Track Indexed timestamp across consuming segments URL: https://github.com/apache/incubator-pinot/pull/3979#discussion_r266506853 ## File path: pinot-common/src/main/java/org/apache/pinot/common/response/broker/BrokerResponseNati