eric-maynard commented on PR #273: URL: https://github.com/apache/polaris/pull/273#issuecomment-2767681581
@singhpk234 > We change the BasePersistence / PolarisMetaStoreManager for this, are all stakeholders onboard for this ? I think these interfaces probably must change in order to support efficient pagination, but I leave it to the reviewers here to determine if the feature is worth the changes. > Are Entity-Ids assumed to be monotonically increasing ? is this a requirement from Polaris ? As entity_id just needs to unique from what i am aware. They are not assumed to be monotonically increasing by this design, and it's not a requirement. In the EclipseLink store here, we sort by entity ID and return them in that order because it's durable against renames unlike sorting by identifier. But we do not care if some new table is created at a lower entity ID than the current "tail" of the paginated listing operation. It's okay to miss tables that are created after the listing operation is initiated. It's not okay to double-count a table or to miss a table that is actually present during the list. -- 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]
