phillipleblanc commented on issue #2071: URL: https://github.com/apache/datafusion-ballista/issues/2071#issuecomment-4997818802
> It definitely makes sense to capture proto version, i just wonder if we need to capture datafusion version. > > ballista major version tracks datafusion upgrade, we might have background compatibility in ballista, but datafusion proto might be incompatible, so executors might fail to decode proto. do we need to take extra care when ballista is released for datafusion proto compatibility ? We could take the conservative approach of bumping the BALLISTA_PROTOCOL_VERSION on every DF upgrade. -- 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]
