add support for Vertica

justwatchcom / sql_exporter

Flexible SQL Exporter for Prometheus.

MIT License

393 stars 108 forks source link

add support for Vertica #45

Closed lukas-mi closed 2 years ago

lukas-mi commented 4 years ago

Added Vertica driver to the dependencies. Tested on Vertica cluster.

dewey commented 2 years ago

Hey @lukas-mi, is this still something you are interested in? If you are, great! I just took over the maintenance a bit so I'm working my way through the backlog. Sorry for the delay.

There's some small changes I'd like to see.

Please rebase it on the latest master
Add an example connection string to the example file: https://github.com/justwatchcom/sql_exporter/blob/master/config.yml.dist
Add Vertica as supported platform to the README

Thanks!

bentzimortaboola commented 2 years ago

Vertica latest go driver version is 1.2.1 , maybe better to apply latest

bentzimortaboola commented 2 years ago

I have verified and it working fine

dewey commented 2 years ago

@bentzimortaboola Thanks, let's see if the PR gets updated. Otherwise I'll have to take a look in a few days.

lukas-mi commented 2 years ago

Updated the PR @dewey

dewey commented 2 years ago

@lukas-mi Thanks a lot for the contribution! I'm going to prepare a new release after that is in master.