Skip to content

Commit 21a21d1

Browse files
committed
Docs - Pinot - Fix default values and typo
1 parent 68664d1 commit 21a21d1

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

docs/src/main/sphinx/connector/pinot.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,16 +36,16 @@ This can be the ip or the FQDN, the url scheme (`http://`) is optional.
3636
|--------------------------------------------------------|----------| ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
3737
| `pinot.controller-urls` | Yes | A comma separated list of controller hosts. If Pinot is deployed via [Kubernetes](https://kubernetes.io/) this needs to point to the controller service endpoint. The Pinot broker and server must be accessible via DNS as Pinot returns hostnames and not IP addresses. |
3838
| `pinot.broker-url` | No | A host and port of broker. If broker URL exposed by Pinot controller API is not accessible, this property can be used to specify the broker endpoint. Enabling this property will disable broker discovery. |
39-
| `pinot.connection-timeout` | No | Pinot connection timeout, default is `15s`. |
39+
| `pinot.connection-timeout` | No | Pinot connection timeout, default is `1m`. |
4040
| `pinot.metadata-expiry` | No | Pinot metadata expiration time, default is `2m`. |
4141
| `pinot.controller.authentication.type` | No | Pinot authentication method for controller requests. Allowed values are `NONE` and `PASSWORD` - defaults to `NONE` which is no authentication. |
4242
| `pinot.controller.authentication.user` | No | Controller username for basic authentication method. |
4343
| `pinot.controller.authentication.password` | No | Controller password for basic authentication method. |
4444
| `pinot.broker.authentication.type` | No | Pinot authentication method for broker requests. Allowed values are `NONE` and `PASSWORD` - defaults to `NONE` which is no authentication. |
4545
| `pinot.broker.authentication.user` | No | Broker username for basic authentication method. |
4646
| `pinot.broker.authentication.password` | No | Broker password for basic authentication method. |
47-
| `pinot.max-rows-per-split-for-segment-queries` | No | Fail query if Pinot server split returns more rows than configured, default to `2,147,483,647`. |
48-
| `pinot.prefer-broker-queries` | No | Pinot query plan prefers to query Pinot broker, default is `true`. |
47+
| `pinot.max-rows-per-split-for-segment-queries` | No | Fail query if Pinot server split returns more rows than configured, default to `2,147,483,646`. |
48+
| `pinot.prefer-broker-queries` | No | Pinot query plan prefers to query Pinot broker, default is `false`. |
4949
| `pinot.forbid-segment-queries` | No | Forbid parallel querying and force all querying to happen via the broker, default is `false`. |
5050
| `pinot.segments-per-split` | No | The number of segments processed in a split. Setting this higher reduces the number of requests made to Pinot. This is useful for smaller Pinot clusters, default is `1`. |
5151
| `pinot.fetch-retry-count` | No | Retry count for retriable Pinot data fetch calls, default is `2`. |
@@ -226,7 +226,7 @@ Aggregate function pushdown is enabled by default, but can be disabled with the
226226
catalog property `pinot.aggregation-pushdown.enabled` or the catalog session
227227
property `aggregation_pushdown_enabled`.
228228

229-
A `count(distint)` pushdown may cause Pinot to run a full table scan with
229+
A `count(distinct)` pushdown may cause Pinot to run a full table scan with
230230
significant performance impact. If you encounter this problem, you can disable
231231
it with the catalog property `pinot.count-distinct-pushdown.enabled` or the
232232
catalog session property `count_distinct_pushdown_enabled`.

0 commit comments

Comments
 (0)