[BEAM-11805] Replace user-agent for spanner

[allenpradeep]

This patch contains the workaround to replace the user-agent for spanner.
R: @TheNeuralBit @nielm

---

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

• Choose reviewer(s) and mention them in a comment (R: @username).
• Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
• Update CHANGES.md with noteworthy changes.
• If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Post-Commit Tests Status (on master branch)

Lang	SDK	Dataflow	Flink	Samza	Spark	Twister2
Go		---		---		---
Java
Python				---		---
XLang				---		---

Pre-Commit Tests Status (on master branch)

---	Java	Python	Go	Website	Whitespace	Typescript
Non-portable
Portable	---		---	---	---	---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

[allenpradeep]

@TheNeuralBit - This patch has the following checkstyle violation. Can you help with this?
[ant:checkstyle] [ERROR] /usr/local/google/home/alxavier/spannerio/apbeam/sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/spanner/SpannerAccessor.java:35: You are using raw guava, please use vendored guava classes. [ForbidNonVendoredGuava]

[nielm]

@TheNeuralBit - This patch has the following checkstyle violation. Can you help with this?
[ant:checkstyle] [ERROR] /usr/local/google/home/alxavier/spannerio/apbeam/sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/spanner/SpannerAccessor.java:35: You are using raw guava, please use vendored guava classes. [ForbidNonVendoredGuava]

Replace the import
com.google.common.util.concurrent.ThreadFactoryBuilder
With
org.apache.beam.vendor.guava.v26_0_jre.com.google.common.util.concurrent.ThreadFactoryBuilder

[allenpradeep]

Run Java PreCommit

[allenpradeep]

@nielm @thiagotnunes - Thanks

Please take a look.
I have fixed the checkstyle issues and added the support for checking emulator connection.

[TheNeuralBit]

@allenpradeep could you also cherry-pick the the version update commits from #13765? They were rolled back in #13792. I'd like to apply both changes at once to make sure they pass CI before merging.

[allenpradeep]

Run Java PreCommit

[allenpradeep]

@allenpradeep could you also cherry-pick the the version update commits from #13765? They were rolled back in #13792. I'd like to apply both changes at once to make sure they pass CI before merging.

There were some spanner test issues with the previous commit. I'll cherry pick #13765 after the integration tests passes and resend.

[allenpradeep]

@allenpradeep could you also cherry-pick the the version update commits from #13765? They were rolled back in #13792. I'd like to apply both changes at once to make sure they pass CI before merging.

@TheNeuralBit - cherry-picked the the 2 commits from #13765.
Waiting for the tests to complete.

[allenpradeep]

Run Java PreCommit

[codecov]

Codecov Report

Merging #13990 (a88a034) into master (8f87957) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #13990   +/-   ##
=======================================
  Coverage   82.94%   82.94%           
=======================================
  Files         469      469           
  Lines       58343    58343           
=======================================
+ Hits        48394    48395    +1     
+ Misses       9949     9948    -1     

Impacted Files	Coverage Δ
.../python/apache_beam/transforms/periodicsequence.py	96.49% <0.00%> (-1.76%)	⬇️
...ks/python/apache_beam/runners/worker/sdk_worker.py	89.69% <0.00%> (+0.15%)	⬆️
sdks/python/apache_beam/io/iobase.py	84.81% <0.00%> (+0.26%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8f87957...a88a034. Read the comment docs.

[allenpradeep]

Run Java PreCommit

[allenpradeep]

Run Java PostCommit

[TheNeuralBit]

@allenpradeep The merge conflict looks to be due to #13804, which upgraded guava. I don't think it's an actual conflict, it's just that 007c0ab is upgrading gRPC, which is on an adjacent line. The resolution should be to just make sure we upgrade both.

[allenpradeep]

@allenpradeep The merge conflict looks to be due to #13804, which upgraded guava. I don't think it's an actual conflict, it's just that 007c0ab is upgrading gRPC, which is on an adjacent line. The resolution should be to just make sure we upgrade both.

@TheNeuralBit
Done. PTAL

[allenpradeep]

Run Java PostCommit

[TheNeuralBit]

Thank you @allenpradeep, I will merge when CI is green.

[TheNeuralBit]

Failure is a flake in FhirIOReadIT. I'll go ahead and merge.