snazy commented on PR #856: URL: https://github.com/apache/polaris/pull/856#issuecomment-2622491724
> > I thought we do not want to have two PRs #906 and #856 for the same thing (see #884 and https://lists.apache.org/thread/bcnh1dwgoxd2dvtqk6z935gfzmh4q0jq) > > This PR depends on #808, while #808 also depends on this one, creating a cyclic dependency. To resolve this, we have two options: > > 1. Merge [Separate Polaris-native API from Iceberg Rest Catalog API spec #906](https://github.com/apache/polaris/pull/906) first, allowing [Catalog Spec: Policy Management Apis #808](https://github.com/apache/polaris/pull/808) to be based on it. In this case, we can close this PR. > > 2. Merge [Catalog Spec: Policy Management Apis #808](https://github.com/apache/polaris/pull/808) first, then revisit and proceed with this PR afterward. We can close [Separate Polaris-native API from Iceberg Rest Catalog API spec #906](https://github.com/apache/polaris/pull/906) as we don't need it. > > > @snazy @HonahX which one do you prefer? The titles of the PRs are the same so it's the same thing, or isn't it? -- 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]
