Skip to content

feat(pubsub): add support for cloud storage ingestion topics#10959

Merged
hongalex merged 4 commits into
googleapis:mainfrom
hongalex:feat-pubsub-storage-ingestion
Oct 8, 2024
Merged

feat(pubsub): add support for cloud storage ingestion topics#10959
hongalex merged 4 commits into
googleapis:mainfrom
hongalex:feat-pubsub-storage-ingestion

Conversation

@hongalex

@hongalex hongalex commented Oct 7, 2024

Copy link
Copy Markdown
Member

Adds support for Cloud Storage ingestion in topic create/update.

@hongalex hongalex requested a review from shollyman as a code owner October 7, 2024 22:09
@hongalex hongalex requested review from a team October 7, 2024 22:09
@product-auto-label product-auto-label Bot added the api: pubsub Issues related to the Pub/Sub API. label Oct 7, 2024
Comment thread pubsub/topic.go Outdated
Comment thread pubsub/topic_test.go Outdated
t.Errorf("\ngot %+v\nwant %+v", config2.IngestionDataSourceSettings, settings)
}

// Clear schema settings.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

"Clear ingestion settings."

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fixed

Comment thread pubsub/topic.go
@michaelpri10

Copy link
Copy Markdown

LGTM

@hongalex hongalex merged commit 1a11675 into googleapis:main Oct 8, 2024
@hongalex hongalex deleted the feat-pubsub-storage-ingestion branch December 10, 2025 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: pubsub Issues related to the Pub/Sub API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants