[OAK-6391] With FastQuerySize, getSize() returns -1 if there are exactly 21 rows

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Fix Version/s: 1.7.3, 1.0.39, 1.2.27, 1.4.17, 1.6.3, 1.8.0
• Component/s: query
• Assignee: Thomas Mueller
• Reporter: Thomas Mueller
• Created: 23/Jun/17 09:22
• Updated: 08/Oct/19 15:22
• Resolved: 03/Jul/17 09:05

Description

If FastQuerySize is enabled, and the query result has exactly 21 rows, then getSize() returns -1. With 20 or 22 rows, the correct value is returned.

Comment:
One can not assume getSize() always returns a value larger than 0. Sometimes, getSize() returns too many, it could also return too few; it is just an estimate. The loop used to read the rows should only have "it.hasNext() && i < maxCount" as a condition. With maxCount for example 30. If the number of rows read is smaller than 30, then that's the real row count. If it's 30, then you can use getSize() as an estimation. That can still be -1 for "unknown", even with FastQuerySize enabled.

But it's true that -1 is unexpected in this case.

Issue Links

• is duplicated by: OAK-6390 With FastQuerySize, getSize() returns -1 if there are exactly 21 rows