Skip to content

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

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.

2 participants