How to pass `enableArithAbort` to Tedious?

Expected behaviour:

I would like to pass the “enableArithAbort” config option value to Tedious, it makes a lot of difference in my case: the query takes 1 second when it’s on, 1+ minute when it’s off.

Actual behaviour:

I tried the following:

await this.sqlClient.connect({
       ...
        "enableArithAbort": true
      });

But it doesn’t work, enableArithAbort seems to be absent from the config object when it’s checked.

If I manually edit connection.js like this:

    if (true) {
      options.push('set arithabort on');
    } else if (this.config.options.enableArithAbort === false) {
      options.push('set arithabort off');
    }

The option is set correctly and the performance goes through the roof.

Configuration:

Not sure what to put that’s relevant here.

Software versions

$ node --version
v10.15.1

mssql@^5.1.0:
  version "5.1.0"

[deregtd]

This is kinda an issue:

tedious deprecated The default value for `config.options.enableArithAbort` will change from `false` to `true` in the next major version of `tedious`. Set the value to `true` or `false` explicitly to silence this message. webpack:\node_modules\mssql\lib\tedious\connection-pool.js?, <anonymous>:61:23

For those of us trying to use this module on azure, a big ol warning stuffed into our logs every startup is not … thrilling. 😃

[ggb88]

this worked for me

mdPool = new sql.ConnectionPool({
            user: 'sa',
            password: '********',
            server: 'localhost',
            port: 32768,
            database: 'UnityMD',
            options: {
                "enableArithAbort": true
            }
        });