You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2019/03/29 14:12:43 UTC
[GitHub] [incubator-pinot] sunithabeeram commented on issue #4025: Extend
stream interfaces to expose message metadata like ingestion timestamp
sunithabeeram commented on issue #4025: Extend stream interfaces to expose message metadata like ingestion timestamp
URL: https://github.com/apache/incubator-pinot/pull/4025#issuecomment-478012157
>@mcvsubbu it’s about changing the indexsegment interface which now depends on stream package and stream package depends on core Segment package
@kishoreg, @mcvsubbu : I can update the index() method to just take in the values we need (in this case ingestion timestamp, instead of stream-metadata. I had it at that object level to allow for easy future extensions, but we can always pass in specific parameters.
----------------------------------------------------------------
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:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org