zhangyue19921010 commented on PR #12407: URL: https://github.com/apache/hudi/pull/12407#issuecomment-2519056459
> Hi @zhangyue19921010 , thanks for picking this up! The update is informative amd the cache storage separation is very insteresting and it's obviously performant as well. > > I've left some comments, and below are some high-level suggestions: > > * We can omit the implementation details and focus on the `StorageStrategy` abstraction in this RFC > * Please also update the case for `Object Storage Strategy`, I'm afraid some of the assumptions we made back then may not hold true anymore and will need an update > * I remember we discussed offline that we want to keep a map in `HoodieStorage` to track storage locations on different storages, but I don't see it in this PR. Is that merged into the idea of storing prefixes in metadata table? Comments are all addressed. PTAL! -- 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]
