Skip to content

feat: Add error handling and per-table metrics #20909

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
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat: Add error handling and per-table metrics #20909

wants to merge 2 commits into from
+45 −23

Conversation

przste-go
Copy link
Contributor

@przste-go przste-go commented Jun 25, 2025 •
edited
Loading

Summary

⚠️ If you're contributing to a plugin please read this section of the contribution guidelines 🧑‍🎓 before submitting this PR ⚠️

This PR adds handling of error messages from source plugins and add new statistics for platform calls. This solves two things:

  1. Having proper error statistics when using plugins in development mode
  2. Adding ability to show detailed per table metrics in platform without need to rely on otel_traces

@przste-go przste-go marked this pull request as ready for review June 27, 2025 14:13
@przste-go przste-go requested a review from a team as a code owner June 27, 2025 14:13
@przste-go przste-go requested a review from blesniewski June 27, 2025 14:13
@przste-go przste-go force-pushed the feature/eng-1505-explore-possibility-of-replacing-otel-with-cli-progress branch from 611db55 to 57ad38b Compare June 30, 2025 14:29
murarustefaan
murarustefaan reviewed Jul 1, 2025
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@murarustefaan murarustefaan murarustefaan approved these changes

@blesniewski blesniewski Awaiting requested review from blesniewski blesniewski is a code owner automatically assigned from cloudquery/integrations-backend

Assignees
No one assigned
Labels
area/cli
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@przste-go @murarustefaan @cq-bot