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

openlineage, aws: add OpenLineage support for Redshift SQL. #35794

Merged

Conversation

JDarDagran
Copy link
Contributor

Add flat information schema query support in SQLParser.

This PR adds support for Redshift SQL.

Additional changes in OpenLineage SQL utils were neccessary. This is because Redshift has capability to make cross-database queries, however, traditional cross-database queries to information_schema.columns does not work. Instead one should read from SVV_REDSHIFT_COLUMNS which is a single table containing multi-database information.

@vincbeck
Copy link
Contributor

@ferruzzi, you might be interested into this one :)

@JDarDagran JDarDagran force-pushed the openlineage/add-redshift-sql-support branch from 9140ea8 to 9bffe5c Compare December 11, 2023 12:04
@JDarDagran JDarDagran requested a review from ferruzzi December 11, 2023 21:34
@ferruzzi
Copy link
Contributor

Still waiting to resolve the conversation with Niko but other than that discussion, the rest LGTM

@JDarDagran
Copy link
Contributor Author

Pinging @o-nikolas then :)

Add flat information schema query support in SQLParser.

Signed-off-by: Jakub Dardzinski <[email protected]>
@JDarDagran JDarDagran force-pushed the openlineage/add-redshift-sql-support branch from 9bffe5c to bedcba1 Compare December 20, 2023 21:17
@JDarDagran
Copy link
Contributor Author

@ferruzzi it seems we're ready to go?

@JDarDagran
Copy link
Contributor Author

@ferruzzi @o-nikolas the PR is hanging for quite a long time. Is there anything in particular that would need explanation or any action?

@o-nikolas o-nikolas merged commit 44b97e1 into apache:main Jan 4, 2024
51 checks passed
@o-nikolas
Copy link
Contributor

@ferruzzi @o-nikolas the PR is hanging for quite a long time. Is there anything in particular that would need explanation or any action?

Dennis gave a LGTM in a previous comment, so I approved and merged with two approvals :)

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.

5 participants