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

fix(airflow_db_cleanup): sessions cleanup #11035

Merged
merged 1 commit into from
Jan 17, 2024

Conversation

mokshasoul
Copy link
Contributor

This fixes an issue where the python operator gets killed if the ssession table is large enough by using COUNT(*) instead of loading everything into memory

This fixes an issue where the python operator gets killed if the ssession table is large neough
@mokshasoul mokshasoul requested review from a team as code owners December 15, 2023 11:28
@product-auto-label product-auto-label bot added samples Issues that are directly related to samples. api: composer Issues related to the Cloud Composer API. labels Dec 15, 2023
@meredithslota meredithslota removed their assignment Dec 15, 2023
@leahecole leahecole added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 8, 2024
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 8, 2024
Copy link
Contributor

@apilaskowski apilaskowski left a comment

Choose a reason for hiding this comment

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

Nice catch. Thank you for updating.

Copy link
Contributor

@iennae iennae left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks for the contribution and improving how the sample works.

@iennae iennae merged commit 196be7a into GoogleCloudPlatform:main Jan 17, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: composer Issues related to the Cloud Composer API. samples Issues that are directly related to samples.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants