cursor.read(rowCount) will hang if rowCount larger than total rows' size

[RogerKang]

Assume we have a table which contains 100 rows of data. And we query a select statement with cursor given rowCount = 1000:

cursor.read(1000, (err, rows) => {
  // do something
});

And the callback of read() will never be called.
If we set rowCount <= 100, then everything works as expected.
But we can not always know how many rows inside a table. So I think a cursor should work fine when the total row count less than cursor rowCount parameter.

[charmander]

I’d expect this too, and I think https://github.com/brianc/node-postgres/blob/48f4398fa75247f4ed8e2470372d0b77712f73e3/packages/pg-cursor/test/no-data-handling.js confirms that it’s not intended behaviour.

[brianc]

this shouldn't happen. Can you submit a self-contained js/ts file to reproduce the issue? If so I can fix it. As it stands now we have tests specifically covering this not happening so it might be a user error or something related to your system.