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.

Sign up for GitHub

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

Jump to bottom

Remove deprecated functionality from Google provider #43953

Merged
merged 1 commit into from
Nov 16, 2024
Merged

Remove deprecated functionality from Google provider #43953

merged 1 commit into from
Nov 16, 2024

Conversation

moiseenkov
Copy link
Contributor

Remove deprecated functionality that reached the end of life November 01, 2024.

@moiseenkov moiseenkov force-pushed the remove_deprecated_functionality branch 2 times, most recently from 1b4ad3d to d3ea552 Compare November 13, 2024 10:33
@VladaZakharova
Copy link
Contributor

Hi @potiuk @eladkal @kosteev !
This PR removes operators, hooks and other stuff that was planned for removal after November 1st, so based on our deprecation policy (6 months notice period)

@moiseenkov moiseenkov force-pushed the remove_deprecated_functionality branch 2 times, most recently from 786f4df to 0e33600 Compare November 14, 2024 07:28
@potiuk potiuk requested a review from eladkal November 14, 2024 11:59
potiuk
potiuk approved these changes Nov 14, 2024
View reviewed changes
Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

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

I love those kinds of changes. @eladkal ?

@potiuk
Copy link
Member

potiuk commented Nov 14, 2024

#protm

@potiuk
Copy link
Member

potiuk commented Nov 14, 2024

I think you got big chances for PR of the month @moiseenkov 👍

image

@moiseenkov moiseenkov force-pushed the remove_deprecated_functionality branch 3 times, most recently from 3658d6e to 7c1c4c6 Compare November 15, 2024 06:30
@moiseenkov moiseenkov mentioned this pull request Nov 15, 2024
Closed
@moiseenkov moiseenkov force-pushed the remove_deprecated_functionality branch from 7c1c4c6 to d20fc38 Compare November 15, 2024 09:30
@shahar1 shahar1 self-requested a review November 16, 2024 15:45
shahar1
shahar1 approved these changes Nov 16, 2024
View reviewed changes
Copy link
Contributor

@shahar1 shahar1 left a comment

Choose a reason for hiding this comment

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

IMO - PROTM, less is definitely more in this case :)
Well done!

@shahar1 shahar1 merged commit 3256e43 into apache:main Nov 16, 2024
46 checks passed
kandharvishnu pushed a commit to kandharvishnu/airflow that referenced this pull request Nov 19, 2024
@moiseenkov
Remove deprecated functionality from Google provider (apache#43953)
e07049d
@eladkal eladkal mentioned this pull request Nov 24, 2024
Closed
22 tasks
@moiseenkov moiseenkov mentioned this pull request Dec 19, 2024
Closed
LefterisXefteris pushed a commit to LefterisXefteris/airflow that referenced this pull request Jan 5, 2025
@moiseenkov
Remove deprecated functionality from Google provider (apache#43953)
3b688e8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@potiuk potiuk potiuk approved these changes

@kosteev kosteev kosteev approved these changes

@eladkal eladkal eladkal approved these changes

@shahar1 shahar1 shahar1 approved these changes

@ashb ashb Awaiting requested review from ashb ashb is a code owner

@bolkedebruin bolkedebruin Awaiting requested review from bolkedebruin bolkedebruin is a code owner

Assignees
No one assigned
Labels
area:providers area:serialization kind:documentation provider:google Google (including GCP) related issues
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@moiseenkov @VladaZakharova @potiuk @kosteev @eladkal @shahar1