Hi Yufei,

I'm sorry that I missed the naming misalignment during the review of PR
[1589].

Correcting my previous message, I do believe that the source and binary
archives should have the *same* naming pattern.

If the community agrees, I'd be fine with "polaris-1.0.0-incubating.tar.gz"
(source) and "polaris-bin-1.0.0-incubating.tar.gz" (binary).

Cheers,
Dmitri.

On Tue, Jun 24, 2025 at 11:32 AM Yufei Gu <flyrain...@gmail.com> wrote:

> >
> > Zip and tarball binaries for `polaris-server` and `polaris-admin` are
> > missing in the Maven publication.
>
>
>
> > Dist contents [1] have inconsistent file name patterns, for example:
>
>
> * apache-polaris-1.0.0-incubating.tar.gz
> > * polaris-bin-1.0.0-incubating.tgz
>
>
> I believe binary archives should have "apache-polaris" as a name prefix.
>
>
> Dmitri and Robert, PR 1589 [1] combined both binary distribution
> `polaris-server` and `polaris-admin` to a single bin distribution. There
> are no Zip and tarball binaries for `polaris-server` and `polaris-admin`
> any more.
>
> For the binary package naming,
> 1. We clarify in the PR that: Standardized the binary distribution package
> naming to polaris-{version}.tgz and polaris-{version}.zip, following common
> conventions used by other projects (e.g., spark-3.5.5-bin-hadoop3.tgz).
> 2. We also follow the previous naming convention like `polaris-server` and
> `polaris-admin`, which doesn't have `apache`.
>
> I'm really sorry to see these "-1" comments. We spent almost 3 weeks on
> this PR[1]. 93 conversations from people including Dmitri, JB, Robert and
> Alex. And Dmitri approved the PR. Can we respect and at least remember our
> conversation and decisions we made together in the PR?
>
>
> 1. https://github.com/apache/polaris/pull/1589
>
> Yufei
>
>
> On Tue, Jun 24, 2025 at 7:29 AM Dmitri Bourlatchkov <di...@apache.org>
> wrote:
>
> > -1 (binding)
> >
> > Dist contents [1] have inconsistent file name patterns, for example:
> >
> > * apache-polaris-1.0.0-incubating.tar.gz
> > * polaris-bin-1.0.0-incubating.tgz
> >
> > I believe binary archives should have "apache-polaris" as a name prefix.
> >
> > [1]
> >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-1.0.0-incubating
> >
> > Cheers,
> > Dmitri.
> >
> > On Mon, Jun 23, 2025 at 9:25 PM Yufei Gu <flyrain...@gmail.com> wrote:
> >
> > > Hi everyone,
> > >
> > > I propose that we release the following RC as the official
> > > Apache Polaris 1.0.0-incubating release.
> > >
> > > This corresponds to the tag: apache-polaris-1.0.0-incubating-rc0
> > > *
> > >
> > >
> >
> https://github.com/apache/polaris/commits/apache-polaris-1.0.0-incubating-rc0
> > > *
> > >
> > >
> >
> https://github.com/apache/polaris/tree/3e3cd23dfdc261686f3935c8ea2c472b0bd3e8b8
> > >
> > > The release tarball, signature, and checksums are here, which includes
> > both
> > > source code distribution and a single binary distribution:
> > > *
> > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-1.0.0-incubating
> > >
> > > You can find the KEYS file here:
> > > * https://downloads.apache.org/incubator/polaris/KEYS
> > >
> > > Convenience binary artifacts are staged on Nexus. The Maven repository
> > URL
> > > is:
> > > *
> > >
> > >
> >
> https://repository.apache.org/content/repositories/orgapachepolaris-1020/org/apache/polaris/
> > >
> > > Please download, verify, and test.
> > >
> > > Please vote in the next 72 hours.
> > >
> > > [ ] +1 Release this as Apache polaris 1.0.0-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
> > > Yufei
> > >
> >
>

Reply via email to