davidradl commented on code in PR #28:
URL:
https://github.com/apache/flink-connector-http/pull/28#discussion_r2883725715
##########
docs/content.zh/docs/connectors/datastream/http.md:
##########
@@ -72,15 +72,15 @@ These options are specified on the builder using the
setProperty method.
| flink.connector.http.security.cert.server.allowSelfSigned | optional |
Accept untrusted certificates for TLS communication.
|
| flink.connector.http.sink.request.timeout | optional | Sets
HTTP request timeout in seconds. If not specified, the default value of 30
seconds will be used.
|
| flink.connector.http.sink.writer.thread-pool.size | optional | Sets
the size of pool thread for HTTP Sink request processing. Increasing this value
would mean that more concurrent requests can be processed in the same time. If
not specified, the default value of 1 thread will be used. |
-| flink.connector.http.sink.writer.request.mode | optional | Sets
Http Sink request submission mode. Two modes are available to select, `single`
and `batch` which is the default mode if option is not specified.
|
+| flink.connector.http.sink.writer.request.mode | optional | Sets
the Http Sink request submission mode. Two modes are available: `single` and
`batch`. Defaults to `batch` if not specified. |
| flink.connector.http.sink.request.batch.size | optional |
Applicable only for `flink.connector.http.sink.writer.request.mode = batch`.
Sets number of individual events/requests that will be submitted as one HTTP
request by HTTP sink. The default value is 500 which is same as HTTP Sink
`maxBatchSize` |
### Batch submission mode
-By default, batch size is set to 500 which is the same as Http Sink's
`maxBatchSize` property and has value of 500.
-The `maxBatchSize` property sets maximal number of events that will be
buffered by Flink runtime before passing it to Http Sink for processing.
+By default, batch size is set to 500 which is the same as Http Sink's
`maxBatchSize` property.
Review Comment:
I don't understand this sentence before! when we say batch size - I assume
there is a batch size property, if so we should refer to it by property name.
The way it reads is ambiguous as we day the batch size is 500 bit the same as a
configurable property ? I assume we are saying that the default values for both
these batch size configuration properties is 500, but the way it is phrased is
not clear.
--
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]
