XN137 commented on PR #2261:
URL: https://github.com/apache/polaris/pull/2261#issuecomment-3159894091

   > The culprit here is that although all matching entities are known during 
the list-operation, all you get from it is this EntityNameLookupRecord, so you 
have to load the same entities again.
   Having a counterpart of PolarisMetaStoreManager#listEntities that doesn't 
yield the unnecessarily reduced result but the actual entities would help.
   WDYT?
   
   yeah this is mentioned in the commit message and implied by the added TODO.
   but since a fix/change at the persistence layer is more involved, so as a 
first step and immediate replacement for PR-2258 i thought centralizing the 
workaround is useful, as it doesnt prevent or delay the proper fix.


-- 
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: issues-unsubscr...@polaris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to