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-481520442 @mayankshriv: 1. The interface modification has been kept generic enough to allow other metadata to be included from stream->mutable segment. 2. #3979 and other changes will wire this up. The specific classes that implement the MessageBatch or MessageDecoder will eventually provide the values needed. 3. We definitely considered this option - but accessing the information through segment metadata is a lot more cleaner than other options. I have an updated to the other PR - will push it now. You can review that.
---------------------------------------------------------------- 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]
