jayshrivastava commented on code in PR #20416:
URL: https://github.com/apache/datafusion/pull/20416#discussion_r2956620097
##########
datafusion/proto/proto/datafusion.proto:
##########
@@ -860,6 +860,12 @@ message PhysicalExprNode {
// across serde roundtrips.
optional uint64 expr_id = 30;
+ // For DynamicFilterPhysicalExpr, this identifies the shared inner state.
+ // Multiple expressions may have different expr_id values (different outer
Arc wrappers)
+ // but the same dynamic_filter_inner_id (shared inner state).
+ // Used to reconstruct shared inner state during deserialization.
+ optional uint64 dynamic_filter_inner_id = 31;
+
Review Comment:
Removed. Now it's an "internal expr id"
--
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]
