Re: [I] Provide pinot schema when initializing StreamMessageDecoder [pinot]

2024-04-24 Thread via GitHub
Jackie-Jiang closed issue #12521: Provide pinot schema when initializing StreamMessageDecoder URL: https://github.com/apache/pinot/issues/12521 -- 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 s

Re: [I] Provide pinot schema when initializing StreamMessageDecoder [pinot]

2024-02-29 Thread via GitHub
Jackie-Jiang commented on issue #12521: URL: https://github.com/apache/pinot/issues/12521#issuecomment-1972111556 Yes. I think we can modify the javadoc, and it should apply to all decoders. It doesn't make sense to have a decoder that decodes nothing -- This is an automated message from

Re: [I] Provide pinot schema when initializing StreamMessageDecoder [pinot]

2024-02-28 Thread via GitHub
rseetham commented on issue #12521: URL: https://github.com/apache/pinot/issues/12521#issuecomment-1970554202 For clarification, > @param fieldsToRead The fields to read from the source stream. If blank, reads all fields (only for AVRO/JSON currently) So o

Re: [I] Provide pinot schema when initializing StreamMessageDecoder [pinot]

2024-02-28 Thread via GitHub
Jackie-Jiang commented on issue #12521: URL: https://github.com/apache/pinot/issues/12521#issuecomment-1970547053 cc @snleee @swaminathanmanish -- 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 th

[I] Provide pinot schema when initializing StreamMessageDecoder [pinot]

2024-02-28 Thread via GitHub
rseetham opened a new issue, #12521: URL: https://github.com/apache/pinot/issues/12521 [StreamMessageDecoder's](https://github.com/apache/pinot/blob/ac13a191b945a80084f0a2794391e4be2f463252/pinot-spi/src/main/java/org/apache/pinot/spi/stream/StreamMessageDecoder.java#L49) init is `void i