Investigate non-atomic update to graal-sdk dependency in dependencies.properties file

googleapis / sdk-platform-java

Tooling and shared libraries for Cloud SDK for Java

https://cloud.google.com/java/docs/bom

Apache License 2.0

65 stars 52 forks source link

Investigate non-atomic update to graal-sdk dependency in dependencies.properties file #2210

Open mpeddada1 opened 11 months ago

mpeddada1 commented 11 months ago

With java-shared-config:1.6.0, the dependency management of graal-sdk in maven was shifted from sdk-platform-java to java-shared-config. However, for bazel, we still need to make an additional update to the graal-sdk version in sdk-platform-java.

TODO: Investigate if it is feasible to consolidate the updates to the graal-sdk.

alicejli commented 8 months ago

@mpeddada1 I forget - is this still an open question?

mpeddada1 commented 5 months ago

@alicejli thanks for the note. Apologies for missing this earlier. Yes, this still remains to be an issue. While the dependency is atomically updated in Maven, a manual update in bazel is still needed. That being said, modifying the label to feature request since it is a proposed enhancement to how the update is done as opposed to a regression.