Documentation update #217
Documentation update #217
Conversation
Update installation.md
|
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 |
There was a problem hiding this comment.
idk if this will be necessary as it will be added by default.
@vishalya what do you think?
|
The Trino Gateway configuration is separate from the Trino configuration.. we might need both. This PR would have to move the changes into a section that talks about the Trino cluster configuration. And @vishalya is still updating his PR to add the same config to the docker example config and then we will merge that PR separately. |
|
👋 @ruschecker .. are you going to update this PR or should I send a replacement addressing the concerns raised in the review. |
|
Replaced by #297 as agreed in the dev sync meeting. |
This config required to let trino know that request has been proxied from gateway.