Skip to content

feat: Implement Pub/Sub UDF ingestion strategy#445

Draft
jessikad-google wants to merge 5 commits into
mainfrom
jessikad/pubsub_udfs
Draft

feat: Implement Pub/Sub UDF ingestion strategy#445
jessikad-google wants to merge 5 commits into
mainfrom
jessikad/pubsub_udfs

Conversation

@jessikad-google

Copy link
Copy Markdown
Contributor
  • Update event.proto with promoted fields
  • Add ingest_udf.js for payload transformation
  • Configure Pub/Sub subscription in terraform

- Update event.proto with promoted fields
- Add ingest_udf.js for payload transformation
- Configure Pub/Sub subscription in terraform
- Promotes updated_at from pull_request/issue payload
- Enables event-time analytics instead of ingestion-time
- Add comment_id, review_id for distinct counting
- Add pull_request_state, title, created_at, closed_at
- Add issue_is_pr flag
- Only apply Ingestion UDF to the raw/json subscription
- Keeps legacy pipeline untouched
- Matches logic of Submission Stats which focuses on successful merges
- Uses pull_request.merged === true check
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant