dennishuo commented on PR #2290: URL: https://github.com/apache/polaris/pull/2290#issuecomment-3203861099
>>> It's technically possible to add basic loadEntities that preserves this invariant for now without the filter/transformer arguments, while still providing the benefit of the more fundamental functionality of fetching entire entities all at once. >> Both loadEntitiesByName and loadEntitiesById seem like they solve the N+1 problem and also avoid trying to serialize lambdas over the wire. >we could use those but we still would be running 2 queries instead of 1 i.e. for getting all principals. @collado-mike I interpreted your statement to just be an observation that such "batch load" methods would indeed be a useful option while my mention may have been ambiguous between "load entities under parent", "load batch by id", and "load batch by name", rather than a statement *against* a "load entities under parent", is that right? -- 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]
