search

kafkaesque-io / pulsar-helm-chart

Helm Chart for an Apache Pulsar Cluster
https://helm.kafkaesque.io
Apache License 2.0
31 stars 22 forks source link

Updating to 2.7.1 - JVM config for Presto #86

Open Benoye opened 3 years ago

Benoye commented 3 years ago

Hello all !

Concerning the pulsar helm chart

I am trying to run this chart with an updated version of Pulsar, preferrably 2.7.1 Currently, the version is 2.6.x and 2.5.2 for the PulsarSQL

My main problem is that the Byte field of AvroSchema does not work before version 2.6.2 It is fixed by this pull request : https://github.com/apache/pulsar/pull/6925

No problem to update the other images to 2.7.1 but pulsarSQL does not start because it needs one more option in the jvm : -Dpresto-temporarily-allow-java8=true

Indeed, it was added in the official repo of Pulsar from version 2.6.2 : https://github.com/apache/pulsar/blob/v2.7.1/conf/presto/jvm.config

I suggest to add the option in data.jvm.config to these two files

pulsar/templates/presto/configmap-worker.yaml
pulsar/templates/presto/configmap-coordinator.yaml

I am currently testing this change locally, I will keep you posted if it works ! Thanks for this great chart :)

Benoye commented 3 years ago

It worked ! We are running Pulsar and Pulsar-SQL on 2.7.1