Jefffrey commented on code in PR #21031:
URL: https://github.com/apache/datafusion/pull/21031#discussion_r3090352121
##########
datafusion/proto/proto/datafusion.proto:
##########
@@ -981,6 +981,9 @@ message PhysicalBinaryExprNode {
PhysicalExprNode l = 1;
PhysicalExprNode r = 2;
Review Comment:
I don't think we have a stance on maintaining backward compatibility yet; so
far we haven't bothered, though there is this issue raised to discuss the
possibility:
- https://github.com/apache/datafusion/issues/18270
I believe recently DataFusion is trying to cut down on breaking changes so
perhaps it's worth to keep this backwards compatibility with that in mind
--
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]
