Fair point. I agree that the file naming misalignment is not a release
blocker.
However, it would be nice to see what other people think about those file
names since the pattern will probably continue for a long time after 1.0.
I'm not changing my vote due to other concerns raised on this thread.
Cheers,
Dmitri.
On Tue, Jun 24, 2025 at 12:31 PM Yufei Gu <flyrain...@gmail.com> wrote:
> > 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).
>
> I'd not block the 1.0 release for this. WDYT?
>
> Yufei
>
>
> On Tue, Jun 24, 2025 at 9:01 AM Dmitri Bourlatchkov <di...@apache.org>
> wrote:
>
> > 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
> > > > >
> > > >
> > >
> >
>
