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