This is an automated email from the ASF dual-hosted git repository.
jackie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 9b58cd2 Refactor how Pinot catalog config is defined in presto helm
chart (#6829)
9b58cd2 is described below
commit 9b58cd2e9369be6eb4df5cd0d211ed6c78d38b6b
Author: harold-kfuse <77747253+harold-kf...@users.noreply.github.com>
AuthorDate: Thu Apr 22 02:01:17 2021 -0700
Refactor how Pinot catalog config is defined in presto helm chart (#6829)
A catalogs map is introduced in the values.yaml, which specifies
the Pinot catalog config in the config map template.
---
kubernetes/helm/presto/Chart.yaml | 4 ++--
.../helm/presto/templates/coordinator/configmap.yaml | 12 ++++--------
kubernetes/helm/presto/values.yaml | 18 ++++++++++--------
3 files changed, 16 insertions(+), 18 deletions(-)
diff --git a/kubernetes/helm/presto/Chart.yaml
b/kubernetes/helm/presto/Chart.yaml
index 167994f..84b03d3 100644
--- a/kubernetes/helm/presto/Chart.yaml
+++ b/kubernetes/helm/presto/Chart.yaml
@@ -18,10 +18,10 @@
#
apiVersion: v1
-appVersion: 0.2.0
+appVersion: 0.2.1
name: presto
description: Presto is an open source distributed SQL query engine for running
interactive analytic queries against data sources of all sizes ranging from
gigabytes to petabytes.
-version: 0.2.0
+version: 0.2.1
keywords:
- analytics
- database
diff --git a/kubernetes/helm/presto/templates/coordinator/configmap.yaml
b/kubernetes/helm/presto/templates/coordinator/configmap.yaml
index 9dbb34f..df656a2 100644
--- a/kubernetes/helm/presto/templates/coordinator/configmap.yaml
+++ b/kubernetes/helm/presto/templates/coordinator/configmap.yaml
@@ -22,14 +22,10 @@ kind: ConfigMap
metadata:
name: presto-catalog
data:
- pinot.properties: |-
- connector.name={{ .Values.pinot.connectorName }}
- pinot.controller-urls={{ .Values.pinot.controllerUrls }}
- pinot.controller-rest-service={{ .Values.pinot.controllerRestService }}
- pinot.allow-multiple-aggregations={{
.Values.pinot.allowMultipleAggregations }}
- pinot.use-date-trunc={{ .Values.pinot.useDateTrunc }}
- pinot.infer-date-type-in-schema={{ .Values.pinot.inferDateTypeInSchema }}
- pinot.infer-timestamp-type-in-schema={{
.Values.pinot.inferTimestampTypeInSchema }}
+{{- range $name, $catalog := .Values.catalogs }}
+ {{ $name }}: |
+{{- tpl $catalog $ | nindent 4 -}}
+{{- end }}
---
apiVersion: v1
kind: ConfigMap
diff --git a/kubernetes/helm/presto/values.yaml
b/kubernetes/helm/presto/values.yaml
index 0a668cf..21c449b 100644
--- a/kubernetes/helm/presto/values.yaml
+++ b/kubernetes/helm/presto/values.yaml
@@ -24,14 +24,16 @@ image:
tag: latest
pullPolicy: IfNotPresent
-pinot:
- connectorName: pinot
- controllerUrls: pinot-controller:9000
- controllerRestService: pinot-controller:9000
- allowMultipleAggregations: true
- useDateTrunc: true
- inferDateTypeInSchema: true
- inferTimestampTypeInSchema: true
+catalogs:
+ pinot.properties: |-
+ connector.name=pinot
+ pinot.controller-urls=pinot-controller:9000
+ pinot.controller-rest-service=pinot-controller:9000
+ pinot.allow-multiple-aggregations=true
+ pinot.use-date-trunc=true
+ pinot.infer-date-type-in-schema=true
+ pinot.infer-timestamp-type-in-schema=true
+ pinot.use-streaming-for-segment-queries=false
coordinator:
name: coordinator
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
