-
Notifications
You must be signed in to change notification settings - Fork 249
SaaS Connector - Mixpanel (prototype) #3408
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
base: qa
Are you sure you want to change the base?
Conversation
QA Wolf here! As you write new code it's important that your test coverage is keeping up. |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## qa #3408 +/- ##
==========================================
- Coverage 89.76% 88.77% -0.99%
==========================================
Files 437 453 +16
Lines 90097 93422 +3325
==========================================
+ Hits 80873 82935 +2062
- Misses 9224 10487 +1263 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…ating/editing a mixpanel connection in the client
Relevant issue(s)
https://www.notion.so/Add-SaaS-Connections-25891edacf2b802e895aefe0a12e3809
Description
This body of work started off an a PoC of Meltano. Performance wasn't very good, so I pivoted to writing this code in Rust, which increased performance by over an order of magnitude. While this is now production-ready, it's intended to be used only by Quadratic as an initial review. Changes need to be made to use this more broadly among Quadratic users.
Testing considerations
Get api_secret and project_id from Mixpanel.
QA Wolf tests
Tests need to be added
Dependent docs, links, etc.
Documentation needs to be added. Datafusion supports ANSI SQL, it has some unique features/functions: https://datafusion.apache.org/user-guide/sql/index.html
Demo