Skip to content

Conversation

@roaminggypsy
Copy link
Contributor

@roaminggypsy roaminggypsy commented Nov 30, 2025

Issue: #2450

@wfa-reviewable
Copy link

This change is Reviewable

@roaminggypsy roaminggypsy changed the base branch from main to roaminggypsy-event-group-batch-create-internal November 30, 2025 21:42
@roaminggypsy roaminggypsy force-pushed the roaminggypsy-event-group-batch-create-public branch from 9894914 to 1e79c87 Compare November 30, 2025 22:03
@roaminggypsy roaminggypsy linked an issue Dec 1, 2025 that may be closed by this pull request
Copy link
Member

@SanjayVas SanjayVas left a comment

Choose a reason for hiding this comment

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

@SanjayVas reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @roaminggypsy)

Copy link
Collaborator

@stevenwarejones stevenwarejones left a comment

Choose a reason for hiding this comment

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

@stevenwarejones reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @roaminggypsy)

@roaminggypsy roaminggypsy force-pushed the roaminggypsy-event-group-batch-create-internal branch 3 times, most recently from 37a2d95 to 56d68dc Compare December 4, 2025 22:07
@roaminggypsy roaminggypsy force-pushed the roaminggypsy-event-group-batch-create-public branch from 1e79c87 to 53f918d Compare December 4, 2025 22:09
Base automatically changed from roaminggypsy-event-group-batch-create-internal to main December 4, 2025 23:39
@roaminggypsy roaminggypsy force-pushed the roaminggypsy-event-group-batch-create-public branch 2 times, most recently from 46bcfcf to d459ce0 Compare December 4, 2025 23:55
@roaminggypsy roaminggypsy changed the title feat: Implement public EventGroup BatchCreate feat: Implement public EventGroup batch create Dec 5, 2025
@roaminggypsy roaminggypsy force-pushed the roaminggypsy-event-group-batch-create-public branch from 52c1755 to 46bf7d0 Compare December 5, 2025 07:20
# Conflicts:
#	src/main/kotlin/org/wfanet/measurement/kingdom/deploy/gcloud/spanner/SpannerEventGroupsService.kt
#	src/main/kotlin/org/wfanet/measurement/kingdom/service/internal/testing/EventGroupsServiceTest.kt

# Conflicts:
#	src/main/kotlin/org/wfanet/measurement/kingdom/deploy/gcloud/spanner/SpannerEventGroupsService.kt
#	src/main/kotlin/org/wfanet/measurement/kingdom/deploy/gcloud/spanner/writers/BatchCreateEventGroups.kt
#	src/main/kotlin/org/wfanet/measurement/kingdom/service/internal/testing/EventGroupsServiceTest.kt
#	src/main/proto/wfa/measurement/internal/kingdom/event_groups_service.proto
# Conflicts:
#	src/main/kotlin/org/wfanet/measurement/kingdom/service/api/v2alpha/EventGroupsService.kt
#	src/test/kotlin/org/wfanet/measurement/kingdom/service/api/v2alpha/EventGroupsServiceTest.kt
@roaminggypsy roaminggypsy force-pushed the roaminggypsy-event-group-batch-create-public branch from 46bf7d0 to c240829 Compare December 9, 2025 23:17
Copy link
Contributor Author

@roaminggypsy roaminggypsy left a comment

Choose a reason for hiding this comment

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

@roaminggypsy reviewed 2 of 2 files at r2, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @roaminggypsy)

@roaminggypsy roaminggypsy merged commit 99538e7 into main Dec 10, 2025
7 of 9 checks passed
@roaminggypsy roaminggypsy deleted the roaminggypsy-event-group-batch-create-public branch December 10, 2025 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

EventGroup Batch Create implementation

5 participants