Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Port most of the OAUTH2 support from Trino to Presto #24443

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

auden-woolfson
Copy link
Contributor

@auden-woolfson auden-woolfson commented Jan 27, 2025

Not a cherry pick directly however the following commits are part of this
trinodb/trino@3879f45
trinodb/trino@cd3da24
trinodb/trino@dcb6f0b
trinodb/trino@7cdd133
trinodb/trino@8b8b0be
trinodb/trino@15e53ff

Co-authored-by: lukasz-walkiewicz lukasz.walkiewicz@starburstdata.com
Co-authored-by: Nik Hodgkinson nik.hodgkinson@starburstdata.com

Description

These changes were brought into IBMs internal presto fork, some of the router development changes are dependent on them.

Motivation and Context

Impact

Test Plan

Contributor checklist

  • Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.

Release Notes

Please follow release notes guidelines and fill in the release notes below.

== RELEASE NOTES ==

General Changes
* ... :pr:`12345`
* ... :pr:`12345`

Hive Connector Changes
* ... :pr:`12345`
* ... :pr:`12345`

If release note is NOT required, use:

== NO RELEASE NOTE ==

Not a cherry pick directly however the following commits are part of this
trinodb/trino@3879f45
trinodb/trino@cd3da24
trinodb/trino@dcb6f0b
trinodb/trino@7cdd133
trinodb/trino@8b8b0be
trinodb/trino@15e53ff

Co-authored-by: lukasz-walkiewicz <lukasz.walkiewicz@starburstdata.com>
Co-authored-by: Nik Hodgkinson <nik.hodgkinson@starburstdata.com>
@prestodb-ci prestodb-ci added the from:IBM PR from IBM label Jan 27, 2025
Copy link

linux-foundation-easycla bot commented Jan 27, 2025

CLA Not Signed

@auden-woolfson auden-woolfson force-pushed the oauth2_support_forward_fit branch from 89f8c4d to e5f7203 Compare January 27, 2025 23:58
@steveburnett
Copy link
Contributor

Are we going to need any documentation for this feature?

@prestodb-ci
Copy link

@ethanyzhang imported this issue into IBM GitHub Enterprise

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants