Re: [I] Reduce size of `Expr` struct [datafusion]

2025-06-02 Thread via GitHub
alamb closed issue #16199: Reduce size of `Expr` struct URL: https://github.com/apache/datafusion/issues/16199 -- 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

Re: [I] Reduce size of `Expr` struct [datafusion]

2025-05-27 Thread via GitHub
Omega359 commented on issue #16199: URL: https://github.com/apache/datafusion/issues/16199#issuecomment-2913982326 🤔 -- 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 uns

Re: [I] Reduce size of `Expr` struct [datafusion]

2025-05-27 Thread via GitHub
alamb commented on issue #16199: URL: https://github.com/apache/datafusion/issues/16199#issuecomment-2913974924 > I am surprised by the magnitude of the performance uptick by essentially moving something from the stack to the heap. My theory is that the performance difference is not t

Re: [I] Reduce size of `Expr` struct [datafusion]

2025-05-27 Thread via GitHub
Omega359 commented on issue #16199: URL: https://github.com/apache/datafusion/issues/16199#issuecomment-2913900526 I am surprised by the magnitude of the performance uptick by essentially moving something from the stack to the heap. -- This is an automated message from the Apache Git

Re: [I] Reduce size of `Expr` struct [datafusion]

2025-05-27 Thread via GitHub
alamb commented on issue #16199: URL: https://github.com/apache/datafusion/issues/16199#issuecomment-2913646407 Thanks @hendrikmakait ! -- 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 specifi

Re: [I] Reduce size of `Expr` struct [datafusion]

2025-05-27 Thread via GitHub
hendrikmakait commented on issue #16199: URL: https://github.com/apache/datafusion/issues/16199#issuecomment-2913583669 take -- 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.

Re: [I] Reduce size of `Expr` struct [datafusion]

2025-05-27 Thread via GitHub
hendrikmakait commented on issue #16199: URL: https://github.com/apache/datafusion/issues/16199#issuecomment-2913568586 I'd love to take this one! -- 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

Re: [I] Reduce size of `Expr` struct [datafusion]

2025-05-27 Thread via GitHub
alamb commented on issue #16199: URL: https://github.com/apache/datafusion/issues/16199#issuecomment-2913491785 I think this is a good first issue because 1. there is an example / pattern to follow 2. There is evidence that this will make significant improvements -- This is an autom