SiyaoIsHiding commented on code in PR #2037:
URL: 
https://github.com/apache/cassandra-java-driver/pull/2037#discussion_r2369635983


##########
core/src/main/java/com/datastax/oss/driver/api/core/session/SessionBuilder.java:
##########
@@ -861,6 +874,12 @@ protected final CompletionStage<CqlSession> 
buildDefaultSessionAsync() {
       List<String> configContactPoints =
           defaultConfig.getStringList(DefaultDriverOption.CONTACT_POINTS, 
Collections.emptyList());
       if (cloudConfigInputStream != null) {
+        // override request id generator, unless user has already set it
+        if (programmaticArguments.getRequestIdGenerator() == null) {
+          programmaticArgumentsBuilder.withRequestIdGenerator(new 
W3CContextRequestIdGenerator());
+          LOG.debug(
+              "A secure connect bundle is provided, using 
W3CContextRequestIdGenerator as request ID generator.");

Review Comment:
   Bret and Andy and Jane agreed in the Sep 22 meeting that we will change the 
default key to `traceparent` and allow override



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to