feat(datastore): add support for request tags

[rahulmane-goog]

This PR introduced sending request tags for different datastore requests. The tags can be passed using 2 mechanisms:

1. Instance Level tags.
2. Request Level tags.

• RequestOptions is the proto that holds all the instance and request level tags and is a part of all request protos.
• DatastoreOptions allows the ability to set the instance level tags using setRequestTags. Any operation executed via this Datastore client automatically includes these global request tags in the outbound RPC request payload.
• RequestOptionsHelper an internal utility that combines instance-level tags from DatastoreOptions with any request-level RequestOptions protobuf message before building the wire request.
• DatastoreExecutionOptions is a holder object for all different options like ExplainOptions, RequestOptions and ReadOption.
• Overloaded versions for taking in the DatastoreExecutionOptions for each of the following operations:
  • put
  • get
  • fetch
  • delete
  • newTransaction
  • commit
  • rollback
  • allocateId
  • reserveIds
  • runQuery
  • runAggregationQuery
• Each of these overloaded versions copy the request tags into the request protos.

[gemini-code-assist]

Code Review

This pull request introduces support for passing and merging RequestOptions (such as request tags) in Datastore queries and aggregation queries, allowing both instance-level and request-level tags to be propagated. The feedback highlights a critical issue where instance-level tags are ignored when query-level RequestOptions are not explicitly provided in QueryResultsImpl and AggregationQueryRequestProtoPreparer. Additionally, the reviewer suggests simplifying the varargs withRequestTags overload in Datastore by delegating to the List version, and adding a test case to verify that instance-level tags are correctly populated and sent with queries.

[rahulmane-goog]

Please review

[jinseopkim0]

ci / lint is failing. Could you please take a look?

[rahulmane-goog]

ci / lint is failing. Could you please take a look?

Fixed

[lqiu96]

LGTM, thanks for the changes and working with us on this!

[lqiu96]

Last lint issue:

[ERROR] To fix formatting errors, run "mvn com.spotify.fmt:fmt-maven-plugin:format"
[ERROR] Non complying file: /home/runner/work/google-cloud-java/google-cloud-java/java-datastore/google-cloud-datastore/src/main/java/com/google/cloud/datastore/TransactionImpl.java
[ERROR] Non complying file: /home/runner/work/google-cloud-java/google-cloud-java/java-datastore/google-cloud-datastore/src/main/java/com/google/cloud/datastore/ReadOption.java
[ERROR] Non complying file: /home/runner/work/google-cloud-java/google-cloud-java/java-datastore/google-cloud-datastore/src/main/java/com/google/cloud/datastore/DatastoreImpl.java
[ERROR] Non complying file: /home/runner/work/google-cloud-java/google-cloud-java/java-datastore/google-cloud-datastore/src/main/java/com/google/cloud/datastore/execution/AggregationQueryExecutor.java
[ERROR] Non complying file: /home/runner/work/google-cloud-java/google-cloud-java/java-datastore/google-cloud-datastore/src/main/java/com/google/cloud/datastore/RequestOptionsHelper.java
[ERROR] Non complying file: /home/runner/work/google-cloud-java/google-cloud-java/java-datastore/google-cloud-datastore/src/main/java/com/google/cloud/datastore/QueryResultsImpl.java
[ERROR] Non complying file: /home/runner/work/google-cloud-java/google-cloud-java/java-datastore/google-cloud-datastore/src/main/java/com/google/cloud/datastore/Datastore.java
[ERROR] Non complying file: /home/runner/work/google-cloud-java/google-cloud-java/java-datastore/google-cloud-datastore/src/main/java/com/google/cloud/datastore/DatastoreOptions.java
[ERROR] Non complying file: /home/runner/work/google-cloud-java/google-cloud-java/java-datastore/google-cloud-datastore/src/test/java/com/google/cloud/datastore/AbstractDatastoreTest.java
[ERROR] Non complying file: /home/runner/work/google-cloud-java/google-cloud-java/java-datastore/google-cloud-datastore/src/test/java/com/google/cloud/datastore/models/RequestOptionsTest.java
[ERROR] Non complying file: /home/runner/work/google-cloud-java/google-cloud-java/java-datastore/google-cloud-datastore/src/test/java/com/google/cloud/datastore/it/AbstractITDatastoreTest.java

[lqiu96]

/gcbrun

[lqiu96]

/gcbrun