Skip to content
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

Add vertica adapter #97

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

shane-breeze
Copy link

Added an adapter for the Vertica database https://www.vertica.com/. This database is very similar to postgres.

I hope this PR is sufficient. I tried switching out my repo in my lazy.nvim config but I kept getting Mux.GetAdapters: no driver registered for provided type alias when trying to create a new vertica connection. Am I missing something?

@kndndrj
Copy link
Owner

kndndrj commented Apr 23, 2024

Thanks for the contribution. I'll look into the error and let you know.

@kndndrj
Copy link
Owner

kndndrj commented Apr 24, 2024

Hey @shane-breeze, can you try updating dbee and then instead of installing via lua hook, try running go build instead:

# at dbee root:
go build -C dbee/ -o $HOME/.local/share/nvim/dbee/bin/dbee

(substitute the out path with your binary location)

If it works then, this can be merged!

@shane-breeze
Copy link
Author

Hi @kndndrj, sorry for not coming back to this sooner. Just tested your suggestion and it works a treat - managed to run queries on my vertica database in nvim-dbee

@phdah
Copy link
Contributor

phdah commented Feb 14, 2025

Hey @shane-breeze!

The plugin has been stale for some time, but we have actively started doing work again.
Would you mind syncing you branch, and looking over possible needed changes? Me and @MattiasMTS are pushing for a v0.2.0 release, where the focus is integration tests for the existing adapters. See discussion

Then we would be happy to review this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants