Thanks Russell and JB for verification. I did run Claude before publishing this release and verified the artifacts as OK. But it seems we still have some minor typo and LICENSE issues as pointed out.
I will raise a PR for this and cut the new release soon. Thanks, Ajantha On Sun, Mar 8, 2026 at 10:17 PM Jean-Baptiste Onofré <[email protected]> wrote: > -1 (binding) > > I checked: > - source distribution > -- nit: inner folder (iceberg-catalog-migrator) is not necessary > -- signature and checksum are good > -- LICENSE and NOTICE look good > -- ASF header present in all expected files > -- No binary/compiled file found in the source distribution > - uber jar (binary distribution) > -- signature and checksum are good > -- LICENSE is not correct: > --- gradlew is not actually bundles in the jar, so no need to document it > --- Apache Iceberg is documented two times (line 224 and 1007), with a typo > line 1007 (artiifact instead of artifact). > --- bouncycastle documentation contains the Maven coordinates, including > the version. That should be removed. > --- nit: Dropwizard is documented in LICENSE, but it's actually Codahale > Metrics from Dropwizard that is included in the jar. We should update the > documentation in the LICENSE (for clarity). > --- there is a typo line 1203: it should be datanucleus (not DataNucleaus). > --- jaxb is documented by Maven coordinates (line 763), it should just be > javax.xml > --- javax.xml.stream (xml-apis) is missing in the LICENSE > --- ASM is missing in the LICENSE (it's shaded in jersey) > --- nit: jsch should be only the BSD one (no need to mention lines 400, > 401, 402) > --- jaspic 1.0 (javax/security/auth) is missing in the LICENSE (we can > consider in the Tomcat scope like Jasper) > --- jansi (fusesource) is missing in the LICENSE > --- if Eclipse Jetty is documented in the LICENSE, Mortbay Jetty is missing > in the LICENSE. Also there are other Mortbay packages to be checked > (included in the jar): util, xml, log, ... > --- cglib (coming from Guice) is missing in the LICENSE > -- NOTICE is not correct as it should be in sync with the LICENSE file > (NOTICE should contain the NOTICE from cglib) > > Regards > JB > > On Tue, Mar 3, 2026 at 7:54 AM Ajantha Bhat <[email protected]> wrote: > > > Hi everyone, > > > > I propose that we release the following RC as the official > > Apache Polaris Iceberg catalog migrator 1.0.0 release. > > > > * This corresponds to the tag: > > apache-polaris-iceberg-catalog-migrator-1.0.0-rc2 > > * > > > > > https://github.com/apache/polaris-tools/commits/apache-polaris-iceberg-catalog-migrator-1.0.0-rc2/ > > * > > > > > https://github.com/apache/polaris-tools/tree/289ca6d3294b97d46f5f3faa62df6a8dfc32c430 > > > > The release tarball, signature, and checksums are here: > > * > > > > > https://dist.apache.org/repos/dist/dev/polaris/apache-polaris-iceberg-catalog-migrator/1.0.0/ > > > > You can find the KEYS file here: > > * https://downloads.apache.org/polaris/KEYS > > > > Convenience binary artifacts are staged on Nexus. The Maven repositories > > URLs are: > > * > > > https://repository.apache.org/content/repositories/orgapachepolaris-1057/ > > > > Please download, verify, and test. > > > > Please vote in the next 72 hours. > > > > [ ] +1 Release this as Apache Polaris Iceberg catalog migrator 1.0.0 > > [ ] +0 > > [ ] -1 Do not release this because... > > > > Only PMC members 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. > > > > - Ajantha > > >
