feat: add MongoDB driver handshake metadata for Java-based client connections - #39504
feat: add MongoDB driver handshake metadata for Java-based client connections#39504alexbevi wants to merge 2 commits into
Conversation
|
Caution The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased. |
145ae9e to
3b5dd20
Compare
|
Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment |
Signed-off-by: Alex Bevilacqua <alex@alexbevi.com>
3b5dd20 to
ac9bf8a
Compare
Resolves Gradle dependency analysis violation caused by using ConnectionString and MongoDriverInformation classes without explicitly declaring mongodb-driver-core as a dependency. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
|
Assigning reviewers: R: @chamikaramj for label java. Note: If you would like to opt out of this review, comment Available commands:
The PR bot will only process comments in the main thread (not review comments). |
|
Reminder, please take a look at this pr: @chamikaramj |
|
Assigning new set of reviewers because Pr has gone too long without review. If you would like to opt out of this review, comment R: @Abacn for label java. Available commands:
|
Passes
MongoDriverInformationtoMongoClients.create()inMongoDBResourceManagerso that MongoDB server-side telemetry can identify traffic originating from Apache Beam.What changes
it/mongodb/src/main/java/.../MongoDBResourceManager.javaDRIVER_INFOconstant (@VisibleForTesting) built withMongoDriverInformation.builder().driverName("Apache Beam").build().MongoClients.create(connectionString, DRIVER_INFO)when no externalMongoClientis provided. When a client is injected via the@VisibleForTestingconstructor (e.g. in unit tests), the external client is used unchanged.This will appear in mongos/d logs similar to:
See the MongoDB handshake specification for more info.