Github user viirya commented on the issue: https://github.com/apache/spark/pull/13439 Yea. That is right. The problem is whether it is worth doing this change for the memory usage reduction and performance gain. If the change is relatively small (another implementation, not current one), I think it might be worth. Current change is complicated as you said. Code generation seems too much for this. So you think this patch is not worth doing? Or let me update this to another implementation and then you take a look if it is worth?
--- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org