presto
presto copied to clipboard
Use scheme for querying brokers as specified in the PinotConfig
Test plan - (Please fill in how you tested your changes) These changes have been tested in a Pinot Cluster.
== RELEASE NOTES ==
Pinot Changes
* Ensure pinot connector respects the https scheme property when executing a broker query
- :x: - Lars-Kristian Svenøy The commit (e61176ae01ccea7a6e1271045fa6645ceb42535c) is not authorized under a signed CLA. Please click here to be authorized. For further assistance with EasyCLA, please submit a support request ticket.
This is a bugfix for the presto pinot connector. The connector does not respect the
pinot.use-https-for-broker
pinot.use-https-for-proxy
properties when executing a broker (or broker proxy) query.
If more work is needed on this PR, please let me know.
Thanks for the contribution. Could you please sign the CLA?
Thanks for the contribution. Could you please sign the CLA?
I've started the process, thank you
This pull request has been automatically marked as stale because it has not had recent activity. If you'd still like this PR merged, please comment on the task, make sure you've addressed reviewer comments, and rebase on the latest master. Thank you for your contributions!
Closing as this is no longer relevant