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, sftp: add OpenLineage support for SFTPOperator #31360

Merged
merged 1 commit into from
Jul 25, 2023

Conversation

mobuchowski
Copy link
Contributor

This PR adds OpenLineage metadata support for SFTPOperator.

@pankajkoti
Copy link
Member

cc: @utkarsharma2 could you please review this?

Copy link
Contributor

@phanikumv phanikumv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add docs for this? how does a potential user know how to use the openlineage functionality along with sftp-operator?

airflow/providers/sftp/operators/sftp.py Outdated Show resolved Hide resolved
@mobuchowski
Copy link
Contributor Author

@phanikumv

Can we add docs for this? how does a potential user know how to use the openlineage functionality along with sftp-operator?

As discussed with @sunank200 in other PR, I've added description on how OL data returned from here looks like.

As for user-facing per-operator docs, I don't see reason to or what would be in them - the point of OL integration is that it works automatically without user action on particular operator level, as long as user has "global" OL configuration set up - which in most cases is just the transport.

@pankajkoti pankajkoti removed their request for review July 12, 2023 12:22
@mobuchowski mobuchowski force-pushed the openlineage-sftp-operator branch 2 times, most recently from 8ebaf48 to 55fe602 Compare July 20, 2023 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants