gnodet-bot commented on code in PR #29988:
URL: https://github.com/apache/camel/pull/29988#discussion_r4575446651
##########
components/camel-thrift/src/main/java/org/apache/camel/component/thrift/ThriftConfiguration.java:
##########
@@ -156,7 +156,8 @@ public void setPort(int port) {
}
/**
- * Client timeout for consumers
+ * Socket timeout in milliseconds. On a consumer it is the timeout of the
server socket waiting for a client, and on
+ * a producer the timeout of the client socket connecting to the server.
Review Comment:
⚠️ **STE doc-clarity nit (low):** The description is accurate, but the
second sentence runs to 30+ words. Per the project documentation conventions,
descriptions above 25 words should be split into two shorter sentences for
readability.
```suggestion
* Socket timeout in milliseconds. Used by both sides: on a consumer, it
sets the server
* socket timeout waiting for a client; on a producer, it sets the
client socket connect
* timeout to the server.
```
--
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]
