LiaCastaneda commented on PR #22415: URL: https://github.com/apache/datafusion/pull/22415#issuecomment-4509653190
> @LiaCastaneda I don't think serializing dynamic filters in protobuf was ever the intention. The intention w.r.t. serializing dyanamic filters was being able to find all of the ones in a plan / plan subtree, hook into their update() method and brodcast updates across the network. That is something that is not possible without this API. Ah, I thought it was mainly for dynamic filter discovery in the serialization process (sorry, I'm a bit rusty on this). The runtime discovery + update & broadcast use case makes sense. I don't have a strong opinion about keeping it or not -- datafusion-distributed (or datafusion) will need it, but it's a different project and we haven't started the networking work yet (epic: https://github.com/datafusion-contrib/datafusion-distributed/issues/180). -- 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]
