AnuragRaut08 opened a new pull request, #22451: URL: https://github.com/apache/datafusion/pull/22451
## Which issue does this PR close? Part of #22435 ## What changes are included? Adds `try_to_proto` to `HashTableLookupExpr` so it participates in the expression-local serialization pattern introduced in #21929. `HashTableLookupExpr` holds a runtime `Arc<Map>` that cannot be serialized, so `try_to_proto` replaces it with `lit(true)`. This is safe because the filter is a performance optimisation only — `lit(true)` passes all rows and the join produces correct results either way. The centralized arm in `to_proto.rs` remains as a fallback for now. Cleanup can follow in a separate PR once this lands. ## Are these changes tested? Yes — covered by the existing `roundtrip_hash_table_lookup_expr_to_lit` test in `datafusion/proto/tests/cases/roundtrip_physical_plan.rs`. ## Are there any user-facing changes? No. -- 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]
