msawyer-palantir opened a new pull request, #13730:
URL: https://github.com/apache/iceberg/pull/13730
The first commit fixes a crash when trying to use the pipeline async on a
parallel Reactor thread:
```
Caused by: java.lang.IllegalStateException: block()/blockFirst()/blockLast()
are blocking, which is not supported in thread parallel-1
at
reactor.core.publisher.BlockingSingleSubscriber.blockingGet(BlockingSingleSubscriber.java:83)
at reactor.core.publisher.Mono.block(Mono.java:1742)
at
org.apache.iceberg.azure.adlsv2.VendedAdlsCredentialProvider.credentialForAccount(VendedAdlsCredentialProvider.java:78)
at
org.apache.iceberg.azure.adlsv2.VendedAzureSasCredentialPolicy.maybeUpdateCredential(VendedAzureSasCredentialPolicy.java:58)
at
org.apache.iceberg.azure.adlsv2.VendedAzureSasCredentialPolicy.process(VendedAzureSasCredentialPolicy.java:46)
at
com.azure.core.http.HttpPipelineNextPolicy.process(HttpPipelineNextPolicy.java:81)
at
com.azure.storage.common.policy.MetadataValidationPolicy.process(MetadataValidationPolicy.java:45)
at
com.azure.core.http.HttpPipelineNextPolicy.process(HttpPipelineNextPolicy.java:81)
at
com.azure.core.http.policy.AddDatePolicy.lambda$process$0(AddDatePolicy.java:43)
at reactor.core.publisher.MonoDefer.subscribe(MonoDefer.java:45)
... 12 more
```
The second commit fixes some other thread-safety issues I noticed in the
relevant code, which indeed caused failures in our tests.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
