charts
charts copied to clipboard
Fix Server.ExtraConfig indentation
Switch workerExtraConfig indentation to 2 Switch coordinatorExtraConfig indentation to 2
Before
config.properties: |
coordinator=true
node-scheduler.include-coordinator=false
http-server.http.port=8080
query.max-memory=4GB
query.max-memory-per-node=1GB
discovery.uri=http://localhost:8080
someKey=someValue
After
config.properties: |
coordinator=true
node-scheduler.include-coordinator=false
http-server.http.port=8080
query.max-memory=4GB
query.max-memory-per-node=1GB
discovery.uri=http://localhost:8080
someKey=someValue
Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to [email protected]. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla
What does that actually fix? Indent of 4 is perfectly valid as well as far as I am concerned.
@cla-bot check
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Christos Kotsis. This is most likely caused by a git client misconfiguration; please make sure to:
- check if your git client is configured with an email to sign commits
git config --list | grep email - If not, set it up using
git config --global user.email [email protected] - Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails
The cla-bot has been summoned, and re-checked this pull request!
This is a valid fix:
% helm template trino charts/trino --set server.coordinatorExtraConfig=aa | grep -B8 aa
config.properties: |
coordinator=true
node-scheduler.include-coordinator=false
http-server.http.port=8080
query.max-memory=4GB
query.max-memory-per-node=1GB
discovery.uri=http://localhost:8080
aa
Fixed in 10e2060e0b6d75a4cad412a464f5da18ad30b0a7