Hey folks,
I'm having trouble running this test successfully. I see that there was
already an attempt to fix it
<https://github.com/apache/iceberg/commit/ed06c9cad8ecfd6cc1c0b9e11e11d509428ba0db>
but
this is not included in the 1.7.x branch. Furthermore, running this test
against the `main` branch also fails.
Has anyone else run into this issue?
I'm using java 17 on M1
```
➜ iceberg git:(main) java --version
openjdk 17.0.6 2023-01-17 LTS
OpenJDK Runtime Environment Zulu17.40+19-CA (build 17.0.6+10-LTS)
OpenJDK 64-Bit Server VM Zulu17.40+19-CA (build 17.0.6+10-LTS, mixed mode,
sharing)
```
```
./gradlew :iceberg-core:test --tests
"TestHadoopCommits.testConcurrentFastAppends"
> Task :iceberg-core:test
TestHadoopCommits > testConcurrentFastAppends(File) FAILED
org.awaitility.core.ConditionTimeoutException: Condition with Lambda
expression in org.apache.iceberg.hadoop.TestHadoopCommits was not fulfilled
within 10 seconds.
at
app//org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
at
app//org.awaitility.core.CallableCondition.await(CallableCondition.java:78)
at
app//org.awaitility.core.CallableCondition.await(CallableCondition.java:26)
at
app//org.awaitility.core.ConditionFactory.until(ConditionFactory.java:1006)
at
app//org.awaitility.core.ConditionFactory.until(ConditionFactory.java:975)
at
app//org.apache.iceberg.hadoop.TestHadoopCommits.lambda$testConcurrentFastAppends$3(TestHadoopCommits.java:455)
```
Best,
Kevin Liu
On Tue, Jan 28, 2025 at 10:40 AM Jean-Baptiste Onofré <j...@nanthrax.net>
wrote:
> Just to be clear: we are working on new changes on NOTICE/LICENSE that
> would be great to include in 1.7.x.
>
> See https://github.com/apache/polaris/pull/903
>
> I think it would be great to have this for 1.8.0 but also for 1.7.2.
> That's why I voted -1 :)
>
> Thoughts ?
>
> Regards
> JB
>
> On Tue, Jan 28, 2025 at 7:16 PM Jean-Baptiste Onofré <j...@nanthrax.net>
> wrote:
> >
> > -1 (non binding)
> >
> > I actually did a change that we should "partially revert": I changed
> > the NOTICE to use bundle name, which is not optimal.
> >
> > I propose to cancel this rc0 to create a rc1 fixing name in NOTICE.
> >
> > Sorry about that.
> >
> > Regards
> > JB
> >
> > On Sun, Jan 26, 2025 at 9:41 PM Jean-Baptiste Onofré <j...@nanthrax.net>
> wrote:
> > >
> > > Hi everyone,
> > >
> > > I propose that we release the following RC as the official Apache
> > > Iceberg 1.7.2 release.
> > >
> > > The commit ID is c2105b2634becf68b3fdabd0ee6fb0b6e93d4f0c
> > > * This corresponds to the tag: apache-iceberg-1.7.2-rc0
> > > * https://github.com/apache/iceberg/commits/apache-iceberg-1.7.2-rc0
> > > *
> https://github.com/apache/iceberg/tree/c2105b2634becf68b3fdabd0ee6fb0b6e93d4f0c
> > >
> > > The release tarball, signature, and checksums are here:
> > > *
> https://dist.apache.org/repos/dist/dev/iceberg/apache-iceberg-1.7.2-rc0
> > >
> > > You can find the KEYS file here:
> > > * https://downloads.apache.org/iceberg/KEYS
> > >
> > > Convenience binary artifacts are staged on Nexus. The Maven repository
> URL is:
> > > *
> https://repository.apache.org/content/repositories/orgapacheiceberg-1180/
> > >
> > > For the changes, checkout the Milestone:
> > > https://github.com/apache/iceberg/milestone/52?closed=1
> > >
> > > Please download, verify, and test. Please vote in the next 72 hours.
> > >
> > > [ ] +1 Release this as Apache Iceberg 1.7.2
> > > [ ] +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.
> > >
> > > Thanks,
> > > Regards
> > > JB & Fokko
>
