Re: Merge module polaris-quarkus-admin and polaris-quarkus-server

2025-05-14 Thread Yufei Gu
Not at all. Thanks for the review, actually! I think it's a good idea to name it polaris-bin-{version}.tgz or polaris-{version}-bin.tgz, which indicates that it's a binary distribution, not a source distribution. Let me make the change. Yufei On Wed, May 14, 2025 at 2:31 PM Dmitri Bourlatchkov

Re: [HEADS UP] Preparing 0.10.0-beta-incubating-rc3

2025-05-14 Thread Jean-Baptiste Onofré
Yes both is ok. For convenience, having distributions available on dist/download, maven central and by extend on docker hub is a good thing for the users. Especially for Maven Central, it’s very convenient because people can easily create custom distributions using the maven-assembly-plugin for in

Re: [ANNOUNCE] Relational JDBC the new default for persistence

2025-05-14 Thread Prashant Singh
Hey Dmitri, > Would you be able to share any detailed performance test numbers for the new JDBC Persistence implementation? Please find the number of the below which i used in the above setup: Test case | Implementation | Query| Number of requests | Number of successful

Re: Merge module polaris-quarkus-admin and polaris-quarkus-server

2025-05-14 Thread Dmitri Bourlatchkov
Thanks, Yufei! I added some comments in GH despite the "draft" PR status. I hope you do not mind :) Overall, using one distribution archive both for the server and the admin tool makes sense to me. 4. Standardized the binary distribution package naming to polaris-{version}.tgz and polaris-{ver

Re: [HEADS UP] Preparing 0.10.0-beta-incubating-rc3

2025-05-14 Thread Yufei Gu
Thanks JB for the clarification! To make it clear, I assume it's also OK to not publish the binary bundle to maven repo, but only the ASF's official distribution directories(https://dist.apache.org/repos/dist/release/). Yufei On Wed, May 14, 2025 at 12:38 PM Jean-Baptiste Onofré wrote: > To be

Re: [HEADS UP] Preparing 0.10.0-beta-incubating-rc3

2025-05-14 Thread Jean-Baptiste Onofré
To be clear 1. The only required distribution from ASF is source distribution 2. for convenience we publish the binary distributions in both dist/download and Maven Central Maven Repository can host any artifacts, and it's convenient for users. Almost all Apache projects do that and that's total

Re: [HEADS UP] Preparing 0.10.0-beta-incubating-rc3

2025-05-14 Thread Jean-Baptiste Onofré
Hi Yes, that's expected, and we do that in a lot of Apache projects: https://repo1.maven.org/maven2/org/apache/activemq/apache-activemq/6.1.6/ https://repo1.maven.org/maven2/org/apache/karaf/apache-karaf/4.4.7/ etc Regards JB On Wed, May 14, 2025 at 8:00 PM Yufei Gu wrote: > > We published the

Re: [Discuss] Add Policy Privileges and PolicyGrant to Management Spec

2025-05-14 Thread Yufei Gu
+1. Thanks Jonas! Yufei On Tue, May 13, 2025 at 9:56 PM Honah J. wrote: > Hi folks, > > I would like to propose extending the management API specification to add > policy-related privileges and making Policy a valid securable for > "addGrantToCatalogRole" and "revokeGrantFromCatalogRole". This

Re: Merge module polaris-quarkus-admin and polaris-quarkus-server

2025-05-14 Thread Yufei Gu
Hi Folk, Thanks a lot for the discussion. Here is the PR( https://github.com/apache/polaris/pull/1589) to merge the binary distribution while still keeping module polaris-quarkus-admin and polaris-quarkus-server separately. *What’s included in the PR:* 1. Introduced a new module that combines

Re: [HEADS UP] Preparing 0.10.0-beta-incubating-rc3

2025-05-14 Thread Yufei Gu
We published the binary distribution bundle to the Maven repository in RC2. You can see it here: https://repository.apache.org/content/repositories/orgapachepolaris-1012/org/apache/polaris/polaris-quarkus-server/0.10.0-beta-incubating/ Are we supposed to do that? From my understanding, the Maven

[HEADS UP] Preparing 0.10.0-beta-incubating-rc3

2025-05-14 Thread Jean-Baptiste Onofré
Hi folks, I opened all the PRs for 0.10.0-beta-incubating-rc3: * https://github.com/apache/polaris/pull/1573 * https://github.com/apache/polaris/pull/1565 * https://github.com/apache/polaris/pull/1562 * https://github.com/apache/polaris/pull/1561 * https://github.com/apache/polaris/pull/1560 I wi