pax95 commented on a change in pull request #4877:
URL: https://github.com/apache/camel/pull/4877#discussion_r557080830



##########
File path: 
components/camel-google-pubsub/src/main/java/org/apache/camel/component/google/pubsub/GooglePubsubComponent.java
##########
@@ -129,6 +132,15 @@ private Publisher buildPublisher(String topicName) throws 
IOException {
             CredentialsProvider credentialsProvider = 
NoCredentialsProvider.create();
             
builder.setChannelProvider(channelProvider).setCredentialsProvider(credentialsProvider);
         }
+        if (googlePubsubEndpoint.isMessageOrderingEnabled()) {
+            builder.setEnableMessageOrdering(true);
+            if 
(StringHelper.trimToNull(googlePubsubEndpoint.getPubsubEndpoint()) != null) {
+                builder.setEndpoint(googlePubsubEndpoint.getPubsubEndpoint());

Review comment:
       From what I see the endpoint is used by the  description = "Endpoint to 
use with local Pub/Sub emulator."
   when running unittests. Also it set's up a NoCredentialsProvider so I guess 
this is not a 'real' runtime usable property.




----------------------------------------------------------------
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.

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


Reply via email to