LuciferYang commented on PR #54748: URL: https://github.com/apache/spark/pull/54748#issuecomment-4044503675
> > One exception that would be worth making is when the map is a Literal. You already have reusing the hashmap if the map itself doesn't change, which would be the case for a literal map, and the cost of building the hash map once for all rows would probably be worth it even for a single key > > Let me try the test with a threshold value of 0. From the test results, when the threshold is set to 0 (always constructing a hashmap), there is an additional absolute delay of 2 to 3 milliseconds per query. However, it might be more appropriate to make this a configurable option, so that users can control this behavior themselves according to different scenarios. -- 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]
