singhpk234 commented on code in PR #15204:
URL: https://github.com/apache/iceberg/pull/15204#discussion_r2751821061
##########
azure/src/integration/java/org/apache/iceberg/azure/adlsv2/AzuriteContainer.java:
##########
@@ -33,7 +33,7 @@ public class AzuriteContainer extends
GenericContainer<AzuriteContainer> {
private static final int DEFAULT_PORT = 10000; // default blob service port
private static final String DEFAULT_IMAGE =
"mcr.microsoft.com/azure-storage/azurite";
- private static final String DEFAULT_TAG = "3.35.0";
+ private static final String DEFAULT_TAG = "latest";
Review Comment:
I am fine with pinning the version, the thing is there is no automatic way
to bump the version to these containers as its hard coded, it might happen when
azurite actually have this fix we just forget, hence i did this as i feel
server upgrades are independent of client upgrades and we should test on the
latest version of the server ?
considering the scope is just sdk bump i can take this up in a different pr
wdyt ?
--
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]
