fix beam snapshot workflow#3326
Conversation
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
|
Is it possible to find which Beam commit causing this flag need to be added? It sounds like a regression in Beam for Java 17+ |
Will try to find. Thanks. |
Hi @Abacn, pretty sure its because of this PR - apache/beam#37473 with the newer BOM and other dependencies being pulled in. The timing fits because it was merged on Feb 2 around 2:00, while the first failing snapshot in this series was around 7:00 that same day. |
|
Hi @Abacn, please let me know of any other concerns or if this is ready. Thanks. |
We can check in this PR to make test green. Good to reach out to Spanner team notifying the new client may break Java17+ users. |
|
Do we need to call anything out in Beam's CHANGES.md? |
|
Or, maybe we would have to make an announcement in Breaking changes for template release notes, once we pick up the next beam release. |
fix #3322