nozjkoitop commented on PR #16607: URL: https://github.com/apache/druid/pull/16607#issuecomment-2378937264
> @nozjkoitop I think @LakshSingla's comment was to only make changes that affect MSQ, since the bug is seen only there. I think your changes might affect other queries as well. Hi @adarshsanjeev ! The main issue is in `ComplexFieldWriter`, and it's caused by the `ValueSelector` potentially returning a byte[], which the writer doesn't expect. This needs to be handled, and you could address it in either the `writeTo()` method or `serde.toBytes()`. Both of these classes are part of Druid's processing, and honestly, I don’t see any other places that would be more isolated for handling this. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
