using connection.query(...).stream() with promise wrapper

why can’t I use connection.query(...).stream() with promise wrapper? (you get error: TypeError: connection.query(…).stream is not a function)

I’d quite like to be able to do: let q = await connection.query(someSQL); as well as

const s1 = connection.query(veryBiqSqlResult).stream();
s1.on('result', async function(row) {...})

in the same script, but atm I need promise api for the first and standard api for the second…

[sidorares]

maybe we need to wait for async generators to land: http://node.green/#ESNEXT-candidate--stage-3--Asynchronous-Iterators-for-await-of-loops

for await (const row of connection.query(veryBiqSqlResult).stream()) {
  console.log(row);
}

( not sure if I can mix buffering .query() => Promise version with non-buffering detecting that .stream() is added, maybe we need another name: `for await (const row of connection.queryStream(veryBiqSqlResult)) {})

You can access non-promise api from wrapper as connection.connection - see https://github.com/sidorares/node-mysql2/blob/97a88530db72c656c16f071a710701ad2830b38d/promise.js#L55

const [rows] = await connection.query(smallSqlResult);
await new Promise((accept, reject) => {
  const s1 = connection.connection.query(veryBiqSqlResult);
  s1.on('result', function(row) {...})
  s1.on('end', accept);
  s1.on('error', reject);
})

[marnixhoh]

@eyalroth I have no idea… I think the docs are a little unclear on this. In case you’re interested, I have asked a somewhat similar question here: https://github.com/mysqljs/mysql/issues/2523