kndndrj / nvim-dbee

Interactive database client for neovim
GNU General Public License v3.0
755 stars 51 forks source link

feat: databricks adapter #89

Open MattiasMTS opened 7 months ago

MattiasMTS commented 7 months ago

What has changed

bcp-dev-ops commented 5 months ago

since #80 was rejected-- what steps can I take to get this one across the finish line?

kndndrj commented 5 months ago

since #80 was rejected-- what steps can I take to get this one across the finish line?

I just don't see the point of adding a for-loop which can be done on the user side. So from my point of view, let's remove those changes from this PR.

Other than that, I'm not sure what needs to be done on the go side.

Maybe @MattiasMTS can elaborate.

MattiasMTS commented 5 months ago

Ah sorry I totally missed this.

Yea I agree with @kndndrj that the #80 commit will be removed.

I've left some TODOs in the description. Give me some time and I can get this in. @bcp-dev-ops whats the urgency?

MattiasMTS commented 5 months ago

Should be ready for review now @kndndrj.

I refactored some of the core stuff, e.g. getPGStructure since we are re-using this more than just the postgres adapter at this point. Also, extended the candies a bit and added some default icons for stuff. Not sure how elegant this is given that some types only exist for some adapters. This could need some thinking but for now I don't want to create a breaking change in the lua config. Feel free to change the icons I added -> for now we separate e.g. "materialized view" from "view" by colors etc.

@bcp-dev-ops let me know if this works for you.

bcp-dev-ops commented 5 months ago

Works like a charm! Thanks @MattiasMTS

@kndndrj can you consider for merging?

phdah commented 5 months ago

Noticed that there are discrepancies in when the helper functions show, in regards to table vs view. Is this a databricks adapter issue, or a plugin issue?

MattiasMTS commented 2 months ago

@kndndrj any objections on getting this in to unblock #77 (if it is still relevant)