EmilyMatt commented on PR #2850: URL: https://github.com/apache/datafusion-comet/pull/2850#issuecomment-3614549380
> Minor comment nit. Thanks for the fix @EmilyMatt! > > I am incredibly curious: are you using PME with Comet successfully? I have not tried it actually, as for my use case I haven't ever needed encryption^^ But I do prefer using native_datafusion as it proves to be faster, and this just broke 2 cases of mine (1 where I use SMJ, and 1 with Union) In general I think instead of broadcasting the config should just be serialized as part of the operator itself, as this seems to break some important functionality^ -- 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]
