Skip to content

Commit

Permalink
chunk: add option-destination-threaded-batching
Browse files Browse the repository at this point in the history
It includes the batch-lines() and batch-timeout() options, which always
come hand-in-hand.

Some places forgot to include batch-timeout(), but those drivers have
that option. Including option-destination-threaded-batching takes care
of this problem.

Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
  • Loading branch information
alltilla committed Aug 17, 2023
1 parent b877073 commit 3e3df17
Show file tree
Hide file tree
Showing 14 changed files with 20 additions and 57 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,7 @@ The `amqp()` destination has the following options:

{{% include-headless "chunk/option-destination-batch-bytes.md" %}}

{{% include-headless "chunk/option-destination-batch-lines.md" %}}

{{% include-headless "chunk/option-destination-batch-timeout.md" %}}
{{% include-headless "chunk/option-destination-threaded-batching.md" %}}


## body() {#amqp-option-body}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,19 +12,11 @@ The `http` destination of {{% param "product.abbrev" %}} can directly post log m
For details on how this option influences HTTP batch mode, see [http: Posting messages over HTTP without Java]({{< relref "/chapter-destinations/configuring-destinations-http-nonjava/_index.md" >}})



{{% include-headless "chunk/option-destination-batch-lines.md" %}}
{{% include-headless "chunk/option-destination-threaded-batching.md" %}}

For details on how this option influences HTTP batch mode, see [http: Posting messages over HTTP without Java]({{< relref "/chapter-destinations/configuring-destinations-http-nonjava/_index.md" >}})



{{% include-headless "chunk/option-d-b-t-http-spl-sent-pub.md" %}}

For details on how this option influences HTTP batch mode, see [http: Posting messages over HTTP without Java]({{< relref "/chapter-destinations/configuring-destinations-http-nonjava/_index.md" >}})



## body()

| | |
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ The following options are required: `bootstrap-servers()`, `topic()`. Note that
{{< include-headless "chunk/kafka-c-impl-required-options-note.md" >}}


{{% include-headless "chunk/option-destination-batch-lines.md" %}}
{{% include-headless "chunk/option-destination-threaded-batching.md" %}}

{{< include-headless "wnt/n-options-only-effective-for-sync-send(yes).md" >}}

Expand All @@ -28,12 +28,6 @@ If you set `sync-send()` to `"yes"`, the number you specify for `batch-lines()`

{{% /alert %}}



{{% include-headless "chunk/option-destination-batch-timeout.md" %}}

{{< include-headless "wnt/n-options-only-effective-for-sync-send(yes).md" >}}

{{% alert title="Note" color="info" %}}

When setting `batch-timeout()`, consider the value of the `transaction.timeout.ms` Kafka property. If in case of timeout (that is, if {{% param "product.abbrev" %}} does not receive `batch-lines()` amount of messages) the value of `batch-timeout()` exceeds the value of `transaction.timeout.ms`, {{% param "product.abbrev" %}} will not send out messages in time.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -63,9 +63,7 @@ Hazard of data loss! The {{% param "product.abbrev" %}} application does not ver

{{% include-headless "chunk/option-destination-batch-bytes.md" %}}

{{% include-headless "chunk/option-destination-batch-lines.md" %}}

{{% include-headless "chunk/option-destination-batch-timeout.md" %}}
{{% include-headless "chunk/option-destination-threaded-batching.md" %}}

{{< include-headless "chunk/option-destination-frac-digits.md" >}}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,8 @@ The `redis()` destination has the following options:

*Description:* The password used for authentication on a password-protected Redis server. Available in {{% param "product.abbrev" %}} version 3.10 and later.

{{% include-headless "chunk/option-destination-batch-lines.md" %}}
{{% include-headless "chunk/option-destination-threaded-batching.md" %}}

{{% include-headless "chunk/option-destination-batch-timeout.md" %}}

## command() {#redis-option-command}

Expand All @@ -42,9 +41,7 @@ Note the following points when using the `redis()` destination:

{{% include-headless "chunk/option-destination-batch-bytes.md" %}}

{{% include-headless "chunk/option-destination-batch-lines.md" %}}

{{% include-headless "chunk/option-destination-batch-timeout.md" %}}
{{% include-headless "chunk/option-destination-threaded-batching.md" %}}

{{< include-headless "chunk/option-destination-hook.md" >}}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -97,13 +97,11 @@ Note that the time format parameter requires:
{{% include-headless "chunk/option-destination-batch-bytes.md" %}}


{{% include-headless "chunk/option-destination-batch-lines.md" %}}
{{% include-headless "chunk/option-destination-threaded-batching.md" %}}

If an error occurs while sending the messages to the server, {{% param "product.abbrev" %}} will try to resend every message from the batch. If it does not succeed (you can set the number of retry attempts in the `retries()` option), {{% param "product.abbrev" %}} drops every message in the batch.
{{% include-headless "chunk/option-destination-batch-timeout.md" %}}
{{< include-headless "chunk/option-destination-hook.md" >}}
## host() {#riemann-option-host}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,7 @@ The `smtp()` sends email messages using SMTP, without needing external applicati

{{% include-headless "chunk/option-destination-batch-bytes.md" %}}

{{% include-headless "chunk/option-destination-batch-lines.md" %}}

{{% include-headless "chunk/option-destination-batch-timeout.md" %}}
{{% include-headless "chunk/option-destination-threaded-batching.md" %}}

{{% include-headless "chunk/smtp-option-from.md" %}}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,7 @@ This driver sends messages into an SQL database. The `sql()` destination has the

{{% include-headless "chunk/option-destination-batch-bytes.md" %}}

{{% include-headless "chunk/option-destination-batch-lines.md" %}}

{{% include-headless "chunk/option-destination-batch-timeout.md" %}}
{{% include-headless "chunk/option-destination-threaded-batching.md" %}}

## columns() {#sql-option-columns}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,9 +39,7 @@ The `stomp()` destination has the following options:

{{% include-headless "chunk/option-destination-batch-bytes.md" %}}

{{% include-headless "chunk/option-destination-batch-lines.md" %}}

{{% include-headless "chunk/option-destination-batch-timeout.md" %}}
{{% include-headless "chunk/option-destination-threaded-batching.md" %}}

{{< include-headless "chunk/option-destination-hook.md" >}}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,18 +24,11 @@ For details on how this option influences HTTP batch mode, see [http: Posting me



{{% include-headless "chunk/option-destination-batch-lines.md" %}}
{{% include-headless "chunk/option-destination-threaded-batching.md" %}}

For details on how this option influences HTTP batch mode, see [http: Posting messages over HTTP without Java]({{< relref "/chapter-destinations/configuring-destinations-http-nonjava/_index.md" >}})



{{% include-headless "chunk/option-d-b-t-http-spl-sent-pub.md" %}}

For details on how this option influences HTTP batch mode, see [http: Posting messages over HTTP without Java]({{< relref "/chapter-destinations/configuring-destinations-http-nonjava/_index.md" >}})



{{% include-headless "chunk/option-destination-tls-ca-dir.md" %}}

{{% include-headless "chunk/topic-tls-block-http.md" %}}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -46,18 +46,11 @@ For details on how this option influences batch mode, see [http: Posting message



{{% include-headless "chunk/option-destination-batch-lines.md" %}}
{{% include-headless "chunk/option-destination-threaded-batching.md" %}}

For details on how this option influences batch mode, see [http: Posting messages over HTTP without Java]({{< relref "/chapter-destinations/configuring-destinations-http-nonjava/_index.md" >}})



{{% include-headless "chunk/option-destination-batch-timeout.md" %}}

For details on how this option influences batch mode, see [http: Posting messages over HTTP without Java]({{< relref "/chapter-destinations/configuring-destinations-http-nonjava/_index.md" >}})



{{% include-headless "chunk/option-destination-tls-ca-dir.md" %}}

{{% include-headless "chunk/topic-tls-block-http.md" %}}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ In addition, the `sumologic-http()` destination also has the following options.

{{% include-headless "chunk/option-destination-batch-bytes.md" %}}

{{% include-headless "chunk/option-destination-batch-lines.md" %}}
{{% include-headless "chunk/option-destination-threaded-batching.md" %}}

{{% include-headless "chunk/option-destination-tls-ca-dir.md" %}}

Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
---
---

{{% include-headless "chunk/option-destination-batch-lines.md" %}}

{{% include-headless "chunk/option-destination-batch-timeout.md" %}}
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,6 @@
<!-- DISCLAIMER: This file is based on the syslog-ng Open Source Edition documentation https://github.com/balabit/syslog-ng-ose-guides/commit/2f4a52ee61d1ea9ad27cb4f3168b95408fddfdf2 and is used under the terms of The syslog-ng Open Source Edition Documentation License. The file has been modified by Axoflow. -->
{{% alert title="Note" color="info" %}}

The {{% param "product.abbrev" %}} configuration accepts this option with `sync-send()` set to both `"yes"` or `"no"`, but the option will only take effect if you set `sync-send()` to `"yes"`.
The {{% param "product.abbrev" %}} configuration accepts `batch-lines()` with `sync-send()` set to both `"yes"` or `"no"`, but the option will only take effect if you set `sync-send()` to `"yes"`.

{{% /alert %}}

0 comments on commit 3e3df17

Please sign in to comment.