Suspected memory leak when streaming data

[mplantard]

Suspected memory leak when streaming data

[dhensby]

Does this happen if you use tedious directly? I'd like to rule out the memory leak happening outside the library.

[mplantard]

Good point, it seems it does with Tedious too, still the same error with the code below.

I wasn't expecting the stream behavior being implemented at the Tedious level but considering the pace of the logs I'm reconsidering my assumption.

let Connection = require('tedious').Connection;
let Request = require('tedious').Request;

let connection = new Connection(config);

// Setup event handler when the connection is established. 
connection.on('connect', function(err) {

    if(err) console.log('Error: ', err)

    // If no error, then good to go...
    let count = 0;
    let subCount = 0;

    const request = new Request('select * from MyTable', function(err, _rowCount) {
        if(err){
            console.log(err)
        } else {
            connection.close()
        };
    });

    request.on('row', function (_columns) {
        count++;
        subCount++;
        if(subCount >= 10000){
            subCount = 0;
            console.log(count);
        }
    });
    request.on('done', function (_rowCount, _more, _rows) { console.log('Completed') });

    connection.execSql(request);
});

// Initialize the connection.
connection.connect();

[dhensby]

OK - I think this is best raised in the tedious repo, if you can replicate the problem directly there.

[dhensby]

Seeing as this appears to be a problem with the underlying driver, I'm going to close this issue. Feel free to add any related comments here or if there's also a problem in this library, we can re-open it.