-
Notifications
You must be signed in to change notification settings - Fork 113
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
fix: skipping users events for snowpipe streaming #3836
Conversation
c9c93e3
to
b43bbaa
Compare
Test report for this run is available at: https://test-integrations-dev.s3.amazonaws.com/integrations-test-reports/rudder-transformer/3836/test-report.html |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #3836 +/- ##
========================================
Coverage 90.47% 90.48%
========================================
Files 615 615
Lines 32329 32339 +10
Branches 7681 7689 +8
========================================
+ Hits 29251 29261 +10
- Misses 2822 2854 +32
+ Partials 256 224 -32 ☔ View full report in Codecov by Sentry. |
5ab0f1c
to
2e13447
Compare
017029f
to
19988fc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. added minor comments
19988fc
to
c1b39f2
Compare
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving to unblock merge
What are the changes introduced in this PR?
users
events for Snowpipe streaming destination.What is the related Linear task?