sql_exporter
sql_exporter copied to clipboard
free
Add snowflake support
Simply adding support for the Snowflake driver. I hope I added the dependency correectly.
Think we could get this merged soon? I would love to be able to use the official sql_exporter image rather than have to build from source and host my own...
Hi @free, it sounds like there's at least two parties looking for this feature. Need anything else from me to facilitate reviewing this PR?
Hey @free! Any updates here? Would really love to be able to use the hosted sql_exporter image. Please let me know if there is anything you need from me to help get this pushed through and a new release cut!
Hey @free, @burningalchemist. It looks like maybe this repo is losing support, as it has not been updated in quite some time now. Is there anything we can do to help support it? I know this is a widely used tool in various locations, but we are in a place where im guessing most people are forking and managing their own, versus just using the open source version.
What can we do to get some of these PRs approved/merged?
Hey @nomeelnoj, you can take a look at the fork I maintain for the moment: https://github.com/burningalchemist/sql_exporter
It's compatible, has some updates, and I keep using it myself, so happy to look into the issues, if there are any. It might be that at some point we can merge things back to the original repo, or maybe not. 🙂
If this works for you, give it a shot. 👍
@burningalchemist thanks! ill definitely take a look. Seems that the original maintainers of sql_exporter have gone MIA...sadly.
@nomeelnoj feel free to leave a comment, if there's an issue with it. 👍 I updated the latest snowflake driver version recently.
It's a bit tricky to test it with snowflake without creating an account, but I'm happy to look into it if anything. 🙂
why is this not merged? we are currently using it, and works as expected.
