pulsar-helm-chart icon indicating copy to clipboard operation
pulsar-helm-chart copied to clipboard

Updating to 2.7.1 - JVM config for Presto

Open Benoye opened this issue 3 years ago • 1 comments

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 avatar May 06 '21 14:05 Benoye

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

Benoye avatar May 06 '21 14:05 Benoye