-
Notifications
You must be signed in to change notification settings - Fork 76
InfluxDB3Source: new connector #788
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: main
Are you sure you want to change the base?
Conversation
Currently blocked, awaiting Tomas feedback. |
The cleanup is complete, only need to add docs now. I have tested it and it works. A few other questions:
Besides these questions, just need to add connector docs and should be good. |
Otherwise I believe this is good to go, please have a look. I only added some minor things like a key setter and being able to name the outgoing measurement_name column (default |
NOTE: This PR is primarily just me cleaning up code that was written by someone else; I largely tried to leave what they had in place while cleaning up what i could.
Example use: