eric-maynard commented on PR #433: URL: https://github.com/apache/polaris/pull/433#issuecomment-2498760043
Unless I am missing something, this is "safe" if `METADATA_CACHE_MAX_BYTES` is set to a relatively low value, right? In the absence of testing showing that there is measurable performance degradation due to churning large strings in the heap when `METADATA_CACHE_MAX_BYTES` is set to e.g. 10000, I posit that the performance is probably OK for now. Indeed, in the limited testing I have done, performance is actually much better compared to a solution that has no caching and always needs to go to object storage for the metadata. -- 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]
