rows.Close() is not required

[gromas]

pgx/rows.go

Line 471 in b9e2b20

rows.Close()

rows.Close() is not required while we have defer rows.Close() above

[felix-roehrich]

rows.Close() may change the result of rows.Err() so this is not equivalent, rather with this an error may go unnoticed.

[jackc]

rows.Close() may change the result of rows.Err() so this is not equivalent, rather with this an error may go unnoticed.

This is correct. From the docs for Rows.Err():

Err must only be called after the Rows is closed