-
Notifications
You must be signed in to change notification settings - Fork 203
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update wal doc #1917
base: main
Are you sure you want to change the base?
Update wal doc #1917
Conversation
`parallelism` | `uint` | How many parallel batches to write. | 10 | no | ||
`external_labels` | `map(string)` | Labels to add to metrics sent over the network. | | no | ||
Name | Type | Description | Default | Required | ||
----------------------|------------------------------------------------------|-----------------------------------------------------------------|---------|---------- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functional change here is using the word long in retry_backoff and flush_interval
`url` | `string` | Full URL to send metrics to. | | yes | ||
`write_timeout` | `duration` | Timeout for requests made to the URL. | `"30s"` | no | ||
`retry_backoff` | `duration` | How long to wait between retries. | `1s` | no | ||
`max_retry_attempts` | Maximum number of retries before dropping the batch. | `0` | no |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing the type
here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just that one missing column for the data type... otherwise looks good to me.
Update WAL doc and importantly remove reference to password_file since we dont support it, it should be handled by
local.file
or other functionality.