databricks / databricks-sql-nodejs

Databricks SQL Connector for Node.js
Apache License 2.0
24 stars 34 forks source link

Remove limit for sockets per connection to fix "Premature close" error + prepare patch release #217

Closed kravets-levko closed 10 months ago

kravets-levko commented 10 months ago

Fixes databricks/databricks-sql-nodejs#208

kravets-levko commented 10 months ago

maxSockets: Infinity is basically a default value since some very early Node version, I just made it more explicit. In combination with socket timeout, I think Node can deal with it. In the worst case, user will get some error when system limit reached. Also, later we're going to make Agent customizable and allow user to tweak those settings