astronomer-providers icon indicating copy to clipboard operation
astronomer-providers copied to clipboard

Implement OpenLineage extractor for Redshift async operators

Open pankajkoti opened this issue 2 years ago • 12 comments

Implement OpenLineage extractor for Redshift async operators

  • [x] RedshiftSQLOperatorAsync
  • [x] RedshiftDataOperatorAsync

pankajkoti avatar Jun 22 '22 07:06 pankajkoti

Broken plugin: [openlineage.airflow.plugin] Failed to import astronomer.providers.amazon.aws.extractors.redshift.RedshiftAsyncExtractor . Update OL version to 0.10.0 and 0.11.0 and tried the same issue exists. Connect with @pankajkoti and try to resolve this.

rajaths010494 avatar Jul 12 '22 06:07 rajaths010494

Current issue is that with the latest Astro image and recent OpenLineage releases 0.9.0, 0.10.0 & 0.11.0 the plugin load is failing for Redshift Extractor. @rajaths010494 to connect with Maciej (GetInData team) and dig deeper into resolving this issue today.

pankajkoti avatar Jul 13 '22 06:07 pankajkoti

Waiting for PR to be raised from Ol side so that i can review and implement the same in our extractor

rajaths010494 avatar Jul 15 '22 06:07 rajaths010494

We can refer to this page: https://www.notion.so/astronomerio/Developing-OpenLineage-in-Astro-9f30a74c74354f168279c1205f4d89a6 to install local packages

pankajkoti avatar Jul 15 '22 06:07 pankajkoti

Today have a connect with maciej for broken plugin issue.

rajaths010494 avatar Jul 28 '22 06:07 rajaths010494

Draft PR has been raised https://github.com/astronomer/astronomer-providers/pull/561

rajaths010494 avatar Jul 29 '22 06:07 rajaths010494

Test cases are pending

rajaths010494 avatar Aug 01 '22 06:08 rajaths010494

Expected to be completed EOD

bharanidharan14 avatar Aug 01 '22 11:08 bharanidharan14

New version of openlineage-airflow==0.12.0 has been released, will revamp unwanted code and then push the changes

rajaths010494 avatar Aug 05 '22 05:08 rajaths010494

Have incorporated the new openlineage-airflow==0.12.0 code for redshift extractors.

rajaths010494 avatar Aug 09 '22 06:08 rajaths010494

are we ready with the PR for this? It is in draft state at the moment.

phanikumv avatar Aug 17 '22 08:08 phanikumv

Ready for review and i have rebased

rajaths010494 avatar Aug 17 '22 08:08 rajaths010494