vertica / vertica-nodejs

Official native node.js client for the Vertica Analytics Database.
https://www.vertica.com/
Apache License 2.0
13 stars 14 forks source link

Clean up overlap between client properties and connection-parameters properties #80

Closed DMickens closed 2 years ago

DMickens commented 2 years ago

There are a lot of properties for a node.js client. Many of these are new, specific to a Vertica connection. The connection properties really belong in the connection-parameters object within the client and not in both locations, which is messy. We need to decide which properties should be exposed in the Client and which should be in the connection-parameters. Whatever doesn't need exposed in the client, which may be all additional properties we have added.

Once this is done, anywhere that we need connection properties such as when creating a startup message, we need to update to take these from the client's connection-parameters.

jonathanl-bq commented 2 years ago

Resolved as Done.