Skip to content

Comments

GoogleDisplayVideo360SDFtoGCSOperator: add unwrap_single flag#61709

Merged
potiuk merged 1 commit intoapache:mainfrom
Ajay9704:display_vedio
Feb 15, 2026
Merged

GoogleDisplayVideo360SDFtoGCSOperator: add unwrap_single flag#61709
potiuk merged 1 commit intoapache:mainfrom
Ajay9704:display_vedio

Conversation

@Ajay9704
Copy link
Contributor

part of #61284 , I improved the return values of two Google ToGCS operators to provide full gs:// URIs instead of partial paths, making them more consistent and directly usable.
Changes made:
GoogleCalendarToGCSOperator - Now returns complete GCS URIs (e.g. gs://bucket/calendar_id.json) instead of just object names
GoogleDisplayVideo360SDFtoGCSOperator - Now returns proper GCS URIs (e.g., gs://bucket/object_name) instead of bucket/object format
These changes address part of issue #11323 by standardizing the return format to provide more useful destination URIs for users

Used AI for resolving conflicts

@Ajay9704 Ajay9704 requested a review from shahar1 as a code owner February 10, 2026 07:52
@boring-cyborg boring-cyborg bot added area:providers provider:google Google (including GCP) related issues labels Feb 10, 2026
@Ajay9704 Ajay9704 changed the title seperted display vedio seperated display vedio Feb 10, 2026
@Ajay9704
Copy link
Contributor Author

@shahar1
Could you please help me understand the failing checks?
They don’t seem related to the changes made for this issue as previously all were passed successfully , so I want to confirm whether these failures are expected or if there’s something I need to fix on my side.
Please let me know how to proceed.

@shahar1
Copy link
Contributor

shahar1 commented Feb 10, 2026

@shahar1 Could you please help me understand the failing checks? They don’t seem related to the changes made for this issue as previously all were passed successfully , so I want to confirm whether these failures are expected or if there’s something I need to fix on my side. Please let me know how to proceed.

Seems unrelated - you may ignore. I reran the CI.

@Ajay9704
Copy link
Contributor Author

@shahar1 Could you please help me understand the failing checks? They don’t seem related to the changes made for this issue as previously all were passed successfully , so I want to confirm whether these failures are expected or if there’s something I need to fix on my side. Please let me know how to proceed.

Seems unrelated - you may ignore. I reran the CI.

ok sir

@shahar1 shahar1 changed the title seperated display vedio GoogleDisplayVideo360SDFtoGCSOperator: add unwrap_single flag and return full GCS URIs Feb 14, 2026
@shahar1 shahar1 changed the title GoogleDisplayVideo360SDFtoGCSOperator: add unwrap_single flag and return full GCS URIs GoogleDisplayVideo360SDFtoGCSOperator: add unwrap_single flag Feb 14, 2026
@potiuk potiuk merged commit 7b33e9b into apache:main Feb 15, 2026
168 of 170 checks passed
choo121600 pushed a commit to choo121600/airflow that referenced this pull request Feb 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:google Google (including GCP) related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants