Bucket Eventing Phase 1 #21

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

Merged

kaloyan-raev merged 10 commits into main from bucket-eventing-phase-1

Sep 25, 2025

Member

kaloyan-raev commented Aug 28, 2025

Closes #19

kaloyan-raev added 8 commits

August 28, 2025 12:24


          Bucket Eventing Phase 1

ec2d9c5


          Added COGS Analysis and Implementation

992a5e4


          Update disclaimers and out of scope for bucket eventing

b92d522


          Expand S3 event notification examples in documentation

89926d3


          Add TODO for Spanner and Pub/Sub emulator testing

73bd8f1

Added a TODO note regarding local testing support for Spanner and Pub/Sub emulator.


          Update bucket eventing implementation details

ffb9f37


          Test plan


          GCP permissions

41e281a

jtolio approved these changes

View reviewed changes

Member

jtolio left a comment

this is fantastic. thorough, well explained, well researched. good plan!

jenlij reviewed

View reviewed changes

20250828-bucket-eventing-phase-1.md Show resolved Hide resolved

jenlij reviewed

View reviewed changes

20250828-bucket-eventing-phase-1.md Outdated Show resolved Hide resolved

jenlij reviewed

View reviewed changes

20250828-bucket-eventing-phase-1.md Show resolved Hide resolved

jenlij reviewed

View reviewed changes

20250828-bucket-eventing-phase-1.md Show resolved Hide resolved

jenlij reviewed

View reviewed changes

20250828-bucket-eventing-phase-1.md Show resolved Hide resolved

jenlij reviewed

View reviewed changes

20250828-bucket-eventing-phase-1.md Show resolved Hide resolved

jenlij approved these changes

View reviewed changes

Contributor

jenlij left a comment

Left some questions but overall looks ready to go.

mobyvb reviewed

View reviewed changes

20250828-bucket-eventing-phase-1.md Show resolved Hide resolved

mobyvb reviewed

View reviewed changes

20250828-bucket-eventing-phase-1.md Show resolved Hide resolved

kaloyan-raev added 2 commits

September 24, 2025 20:45


          typos and grammar

64872d9


          watch for UPDATE records too

07c3a15

kaloyan-raev merged commit ef864ef into main

kaloyan-raev deleted the bucket-eventing-phase-1 branch

September 25, 2025 09:08

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet