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

Add Deferrable Mode to DataplexGetDataQualityScanResultOperator and DataplexRunDataQualityScanOperator #45877

Open
2 tasks done
mahesh989 opened this issue Jan 22, 2025 · 2 comments
Assignees
Labels
area:async-operators AIP-40: Deferrable ("Async") Operators kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet provider:google Google (including GCP) related issues

Comments

@mahesh989
Copy link

Description

The DataplexRunDataQualityScanOperator and DataplexGetDataQualityScanResultOperator are now in non-deferrable mode during operation. This results in the operators using resources excessively when doing lengthy Dataplex API operations. Waiting for API answers causes inefficient use of resources and slows down the completion of other actions in workflows.

Adding a deferrable mode to these operators will allow them to pause execution during wait periods, freeing up resources for other tasks. This change will optimize resource utilization and improve overall workflow performance.

Use case/motivation

Use Case

  1. ** Efficient Long-Running Workflows**: In workflows where tasks interact with the Dataplex API for extended durations, deferrable mode will significantly reduce resource usage, enabling better performance.
  2. Scalable Workflows: Deferrable mode will help users scale their workflows by making better use of available infrastructure.

Motivation

  • Pause and Free Resources while waiting for the Dataplex API to complete its operations.
  • Seamlessly Resume Execution once the API task is complete, without any manual intervention.

Related issues

There doesn't appear to be another linked issue regarding functionality addressed by this PR.

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@mahesh989 mahesh989 added kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet labels Jan 22, 2025
Copy link

boring-cyborg bot commented Jan 22, 2025

Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.

@dosubot dosubot bot added area:async-operators AIP-40: Deferrable ("Async") Operators provider:google Google (including GCP) related issues labels Jan 22, 2025
@potiuk
Copy link
Member

potiuk commented Jan 22, 2025

Feel free to work on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:async-operators AIP-40: Deferrable ("Async") Operators kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet provider:google Google (including GCP) related issues
Projects
None yet
Development

No branches or pull requests

2 participants