Quick update:
1. I opened https://github.com/apache/polaris/pull/1493 to fix the
server distribution "classpath" and include README.md in both admin
and server distributions
2. For the admin, we have to pass $args to run.sh (in order to be able
to do ./run.sh foo). I will push that in #1493 PR too.

Regards
JB

On Tue, Apr 29, 2025 at 2:44 PM Jean-Baptiste Onofré <j...@nanthrax.net> wrote:
>
> Hi
>
> OK, I found the problem and it's not "only" related to the change on
> shipping eclipselink/JDBC driver (the problem occured before that
> using building with eclipselink).
>
> The problem is about the distribution exclude:
> https://github.com/apache/polaris/blob/main/quarkus/server/build.gradle.kts#L94
>
> I'm opening the PR and I will ping Alex (as he did this change).
>
> Regards
> JB
>
> On Tue, Apr 29, 2025 at 9:06 AM Jean-Baptiste Onofré <j...@nanthrax.net> 
> wrote:
> >
> > Hi,
> >
> > The problem has been introduced by the "Ship eclipselink and
> > PostgreSQL JDBC driver by default in Polaris distribution (#1411)".
> > My bad, I will fix that.
> >
> > Regards
> > JB
> >
> > On Tue, Apr 29, 2025 at 2:47 AM Yufei Gu <flyrain...@gmail.com> wrote:
> > >
> > > -1
> > > Issue 1: For server binary distribution, we will need to package the
> > > following 3 jars into polaris-quarkus-server-0.10.0-beta-incubating.zip
> > > <https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating/polaris-quarkus-server-0.10.0-beta-incubating.zip>
> > >
> > > io.quarkus.quarkus-container-image-3.21.2.jar
> > > io.quarkus.quarkus-container-image-docker-3.21.2.jar
> > > io.quarkus.quarkus-container-image-docker-common-3.21.2.jar
> > >
> > > Otherwise, the server could get started, it failed with the
> > > following messages:
> > > 2025-04-28 17:23:27,274 ERROR [io.qua.run.Application] [,] [,,,] (main)
> > > Failed to start application: java.lang.RuntimeException: Failed to start
> > > quarkus
> > > at io.quarkus.runner.ApplicationImpl.doStart(Unknown Source)
> > > at io.quarkus.runtime.Application.start(Application.java:101)
> > > at
> > > io.quarkus.runtime.ApplicationLifecycleManager.run(ApplicationLifecycleManager.java:119)
> > > at io.quarkus.runtime.Quarkus.run(Quarkus.java:80)
> > > at io.quarkus.runtime.Quarkus.run(Quarkus.java:51)
> > > at io.quarkus.runtime.Quarkus.run(Quarkus.java:144)
> > > at io.quarkus.runner.GeneratedMain.main(Unknown Source)
> > > at
> > > io.quarkus.bootstrap.runner.QuarkusEntryPoint.doRun(QuarkusEntryPoint.java:68)
> > > at
> > > io.quarkus.bootstrap.runner.QuarkusEntryPoint.main(QuarkusEntryPoint.java:36)
> > > Caused by: java.lang.RuntimeException: java.nio.file.NoSuchFileException:
> > > /Users/ygu/Downloads/p0.10/polaris-quarkus-server-0.10.0-beta-incubating/lib/main/io.quarkus.quarkus-container-image-docker-common-3.21.2.jar
> > > at
> > > io.quarkus.bootstrap.runner.JarFileReference.asyncLoadAcquiredJarFile(JarFileReference.java:209)
> > > at
> > > io.quarkus.bootstrap.runner.JarFileReference.withJarFile(JarFileReference.java:147)
> > > at
> > > io.quarkus.bootstrap.runner.JarResource.getResourceURL(JarResource.java:89)
> > > at
> > > io.quarkus.bootstrap.runner.RunnerClassLoader.findResources(RunnerClassLoader.java:266)
> > > at java.base/java.lang.ClassLoader.getResources(ClassLoader.java:1483)
> > > at
> > > org.apache.polaris.extension.persistence.impl.eclipselink.PolarisEclipseLinkPersistenceUnit.classpathResource(PolarisEclipseLinkPersistenceUnit.java:193)
> > >
> > >
> > > Issue 2: For admin tool binary distribution, the file run.sh doesn't work.
> > > We have to run `java -jar quarkus-run.jar --help` to make it work.
> > >
> > > A general suggestion: We will also need to package a README to the binary
> > > distribution for instructions, esp. about how to set up the root
> > > credentials. It isn't a blocker though.
> > >
> > > Yufei
> > >
> > >
> > > On Sun, Apr 27, 2025 at 12:09 AM Jean-Baptiste Onofré <j...@nanthrax.net>
> > > wrote:
> > >
> > > > Hi everyone,
> > > >
> > > > Following the fixes done on RC0, I propose that we release the
> > > > following RC (RC1) as the official Apache Polaris
> > > > 0.10.0-beta-incubating release.
> > > >
> > > > * This corresponds to the tag: apache-polaris-0.10.0-beta-incubating-rc1
> > > > *
> > > > https://github.com/apache/polaris/commits/apache-polaris-0.10.0-beta-incubating-rc1
> > > > *
> > > > https://github.com/apache/polaris/tree/131e891975b315f1f7edf5e08c1b57d7bc66326d
> > > >
> > > > The release tarball, signature, and checksums are here:
> > > > *
> > > > https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating
> > > >
> > > > Helm charts are available on:
> > > > * https://dist.apache.org/repos/dist/dev/incubator/polaris/helm-chart
> > > > NB: the docker images (polaris-server and polaris-admin) will be
> > > > published on DockerHub once release vote passes.
> > > >
> > > > You can find the KEYS file here:
> > > > * https://dist.apache.org/repos/dist/release/incubator/polaris/KEYS
> > > >
> > > > Convenience binary artifacts are staged on Nexus. The Maven
> > > > repositories URLs are:
> > > > *
> > > > https://repository.apache.org/content/repositories/orgapachepolaris-1010/
> > > >
> > > > Please download, verify, and test.
> > > >
> > > > Please vote in the next 72 hours.
> > > >
> > > > [ ] +1 Release this as Apache polaris 0.10.0-beta-incubating
> > > > [ ] +0
> > > > [ ] -1 Do not release this because...
> > > >
> > > > Only PPMC members and mentors have binding votes, but other community
> > > > members are
> > > > encouraged to cast non-binding votes. This vote will pass if there are
> > > > 3 binding +1 votes and more binding +1 votes than -1 votes.
> > > >
> > > > NB: if this vote passes, a new vote has to be started on the Incubator
> > > > general mailing
> > > > list.
> > > >
> > > > Thanks
> > > > Regards
> > > >
> > > > JB
> > > >

Reply via email to