Conversation
nomeelnoj
left a comment
nomeelnoj
left a comment
There was a problem hiding this comment.
I was just about to send in my branch for this same change. Would love to see this merged!
|
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. 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. |
|
Any update on this? Why has it not been merged? |
5 participants
Simply adding support for the Snowflake driver. I hope I added the dependency correectly.