Support specifying database in FirestoreV1ReadFn (resolves #36904) (#36905)

* test signing commit

* Support setting project ID and database ID in FirestoreV1 Reads

* WIP adding database ID to read fn

* Revert "WIP adding database ID to read fn"

This reverts commit 133d5a6.

* Reapply "WIP adding database ID to read fn"

This reverts commit f95d988.

* Update read fn constructors to take project and database

* fix build errors

* Update integration test and BatchGet test

* Update remaining unit tests

* Fix write unit test

* Minor format changes

* update Changes.md

* Update javadoc comment for getFirestoreStub

* Run spotlessApply

* fix changes.md

* Fix changes.md conflict

* Take 3 fix changes.md

Author: pacoavila808

CHANGES.md

@@ -72,7 +72,7 @@
 
 ## New Features / Improvements
 
-* X feature added (Java/Python) ([#X](https://github.com/apache/beam/issues/X)).
+* Support configuring Firestore database on ReadFn transforms (Java) ([#36904](https://github.com/apache/beam/issues/36904)).
 
 ## Breaking Changes
 

sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/firestore/FirestoreStatefulComponentFactory.java

@@ -63,12 +63,10 @@ private FirestoreStatefulComponentFactory() {}
    * <p>The instance returned by this method is expected to bind to the lifecycle of a bundle.
    *
    * @param options The instance of options to read from
+   * @param configuredProjectId The project to target, if null, falls back to value in options.
+   * @param configuredDatabaseId The database to target, if null, falls back to value in options.
    * @return a new {@link FirestoreStub} pre-configured with values from the provided options
    */
-  FirestoreStub getFirestoreStub(PipelineOptions options) {
-    return getFirestoreStub(options, null, null);
-  }
-
   FirestoreStub getFirestoreStub(
       PipelineOptions options,
       @Nullable String configuredProjectId,