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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
