Skip to content
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.

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

cdc: add transformations to create changefeed telemetry #95751

Merged
merged 1 commit into from
Jan 24, 2023

Conversation

jayshrivastava
Copy link
Contributor

@jayshrivastava jayshrivastava commented Jan 24, 2023

Previously, there were no telemetry events emitted to track the usage of changefeed transformations. This change adds the transformation field to the CREATE CHANGEFEED telemetry log event to track that. Example:

{
  "Timestamp": 1674574083953989000, "EventType": "create_changefeed",
  "Description": "CREATE CHANGEFEED INTO 'gcpubsub://testfeed?region=testfeedRegion' AS SELECT b FROM foo",
  "SinkType": "gcpubsub", "NumTables": 1, "Resolved": "no", "Transformation": true
}

Epic: CRDB-17161
Closes: #95126

Release note: None

@cockroach-teamcity
Copy link
Member

This change is Reviewable

Previously, there were no telemetry events emitted to track the usage of
changefeed transformations. This change adds the `transformation` field
to the `CREATE CHANGEFEED` telemetry log event to track that. Example:
```
{
  "Timestamp": 1674574083953989000, "EventType": "create_changefeed",
  "Description": "CREATE CHANGEFEED INTO 'gcpubsub://testfeed?region=testfeedRegion' AS SELECT b FROM foo",
  "SinkType": "gcpubsub", "NumTables": 1, "Resolved": "no", "IsTransformation": true
}
```

Epic: CRDB-17161
Closes: cockroachdb#95126

Release note: None
@jayshrivastava jayshrivastava force-pushed the transformations-telemetry branch from 4af35d5 to 974177e Compare January 24, 2023 15:42
@jayshrivastava jayshrivastava marked this pull request as ready for review January 24, 2023 20:19
@jayshrivastava jayshrivastava requested a review from a team January 24, 2023 20:19
@jayshrivastava jayshrivastava requested a review from a team as a code owner January 24, 2023 20:19
@jayshrivastava jayshrivastava requested review from shermanCRL and removed request for a team January 24, 2023 20:19
@jayshrivastava
Copy link
Contributor Author

bors r+

@craig
Copy link
Contributor

craig bot commented Jan 24, 2023

Build succeeded:

@craig craig bot merged commit 64b182c into cockroachdb:master Jan 24, 2023
@jayshrivastava jayshrivastava deleted the transformations-telemetry branch January 26, 2023 15:37
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.

changefeedccl: add telemetry for cdc transformations
3 participants