-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 >
