cloud-fan commented on PR #58975:
URL: https://github.com/apache/spark/pull/58975#issuecomment-6126114250
The gap came from the two components viewing the type differently:
CodeGenerator.isPrimitiveType unwraps a UDT to its SQL type, while
HashMapGenerator.genComputeHash matches the logical data type and has no UDT
case. This patch closes that mismatch at the shared fast-map eligibility check.
<!-- SPARK_DEV_REVIEW_REPLY
{"feedback_id":"conversation:5898188187","thread_id":"conversation:5898188187","verdict_sha256":"e509b074d1681d364081fbec92ec632bb60376cf1e6071971ea053a9a4230a7c"}
-->
--
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]