js8544 commented on PR #40041: URL: https://github.com/apache/arrow/pull/40041#issuecomment-1941117906
Happy Chinese New Year! @pitrou I think this PR is to fix the regression introduced in https://github.com/apache/arrow/pull/12742 rather than a feature. #12742 sets GANDIVA_ENABLE_OBJECT_CODE_CACHE default to true but forget to change the default cache size. @niyue Is my understanding correct? However I do agree that we can further investigate the size limit of the cache. 500000 objects should require too much memory. Suppose one expression takes 1KB (my wild guess), that would be 500MB. -- 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]
