mtxr / vscode-sqltools

Database management for VSCode
https://vscode-sqltools.mteixeira.dev?utm_source=github&utm_medium=homepage-link
MIT License
1.44k stars 291 forks source link

New driver integration - Adapt existing drivers #586

Closed mtxr closed 3 years ago

mtxr commented 4 years ago

Hi @EpicEric, @snyang, @mickeypearce, @ariel-bentu.

Folks, I need your help. I've changed the way we integrate drivers so we can grow without having all drivers bundled with the core extension.

I'm taking advantage of VSCode export extension api feature to allow us to create extensions that can integrate with SQLTools. Now you should be able to publish the drivers by yourselves for example.

That said, can you re-create the drivers and publish extensions for those drivers?

I've updated the docs with some information about it, but the code has some comments as well, also we have for drivers as examples.

Docs: https://vscode-sqltools.mteixeira.dev/contributing/support-new-drivers SQLite: https://github.com/mtxr/vscode-sqltools/tree/master/packages/driver.sqlite MSSQL: https://github.com/mtxr/vscode-sqltools/tree/master/packages/driver.mssql PostgreSQL: https://github.com/mtxr/vscode-sqltools/tree/master/packages/driver.pg MySQL: https://github.com/mtxr/vscode-sqltools/tree/master/packages/driver.mysql

The main idea behind this new release was to have a lazy-tree implementation and some enhancements to completions. Still in the very beginning, but now we should be able to move faster with the features and the integrations.

Feedbacks are welcome.

Let me know if you guys can do those changes

ariel-bentu commented 4 years ago

Hi Metheus, I started porting the code of SAP HANA. The main time consuming task would be to get the legal aspects to publish to vscode marketplace. meanwhile I started the port. I noticed that the generated project does not compile unless I add 4 dev-dependencies: npm install @types/node --save-dev npm install @types/jest --save-dev npm install @types/react-native --save-dev npm install vscode-languageclient --save-dev

Now that it compiles, it outputs to .../out/src/ls the extension.js and you look for it in out/extension.js -> any idea what to change to make it load? Also, At this point the old way still works? so how do I test the new way while the old way for SAP HANA is still in? Thanks, Ariel

On Fri, May 29, 2020 at 7:57 AM Matheus Teixeira notifications@github.com wrote:

Hi @EpicEric https://github.com/EpicEric, @snyang https://github.com/snyang, @mickeypearce https://github.com/mickeypearce, @ariel-bentu https://github.com/ariel-bentu.

Folks, I need your help. I've changed the way we integrate drivers so we can grow without having all drivers bundled with the core extension.

I'm taking advantage of VSCode export extension api feature to allow us to create extensions that can integrate with SQLTools. Now you should be able to publish the drivers by yourselves for example.

That said, can you re-create the drivers and publish extensions for those drivers?

I've updated the docs with some information about it, but the code has some comments as well, also we have for drivers as examples.

Docs: https://vscode-sqltools.mteixeira.dev/contributing/support-new-drivers SQLite: https://github.com/mtxr/vscode-sqltools/tree/master/packages/driver.sqlite MSSQL: https://github.com/mtxr/vscode-sqltools/tree/master/packages/driver.mssql PostgreSQL: https://github.com/mtxr/vscode-sqltools/tree/master/packages/driver.pg MySQL: https://github.com/mtxr/vscode-sqltools/tree/master/packages/driver.mysql

The main idea behind this new release was to have a lazy-tree implementation and some enhancements to completions. Still in the very beginning, but now we should be able to move faster with the features and the integrations.

Feedbacks are welcome.

Let me know if you guys can do those changes

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mtxr/vscode-sqltools/issues/586, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE3ZMN5JYK6EBY5PW5N4VZ3RT46ERANCNFSM4NNWZFVA .

mtxr commented 4 years ago

@ariel-bentu can you re-do the process? I've fixed that already, you should be good to good now with generator-sqltools-driver@v0.1.2

But answering your question, this line is the one that fixed the output issue: https://github.com/mtxr/vscode-sqltools/commit/1410daed8b3619d02ec3b88f9d606f59941fc9d8#diff-79ecd107e18e7dcff3185949efe506b3R5

If I use import, typescript will consider the src folder as a part of the output filename.

Thank you for the effort to port!

stasberkov commented 4 years ago

@EpicEric you did really outstanding job adding support of cassandra 2.x to 0.21 version, please release new driver integration for sqltools 0.22.

EpicEric commented 4 years ago

Adapting driver integration seemed non-trivial, and I don't have enough time to work on this at the moment. Any help would be extremely appreciated.

For now I've been using a previous version of SQLTools.

mickeypearce commented 4 years ago

I am also quite busy at the moment, I will dig into ASAP...

snyang commented 4 years ago

I will get around to dig into.

xgdgsc commented 4 years ago

@EpicEric I saw your working repo on cassandra. How far is it from release?

EpicEric commented 4 years ago

@xgdgsc I still don't have enough time to work on it. I can't give any estimates at this point.

cfont commented 3 years ago

Is the lack of an Oracle Driver simply just needing a developer to have time to work on it or is it more of a legal/license issue?

ElCondor1969 commented 3 years ago

Any updates or roadmap for Oracle Driver porting on last version? Thanks.

mtxr commented 3 years ago

@cfont we need a dev. @mickeypearce has been busy, do you want to try?

mtxr commented 3 years ago

I'm closing this issue since a couple of months passed. In case anyone need an specific driver, feel free to open a issue targeting just the driver needed.

gjsjohnmurray commented 1 year ago

Information about a community-contributed driver for Oracle is now in the table at https://vscode-sqltools.mteixeira.dev/en/drivers/