[DRILL-7414] EVF incorrectly sets buffer writer index after rollover

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.17.0
• Labels: ready-to-commit
• Assignee: Paul Rogers
• Reporter: Paul Rogers
• Reviewer: Arina Ielchiieva
• Created: 20/Oct/19 19:12
• Updated: 22/Oct/19 10:22
• Resolved: 22/Oct/19 10:21

Description

A full test run, with vector validation enabled and with the "new" scan enabled, revealed the following in TestMockPlugin.testSizeLimit():

comments_s20000 - VarCharVector: Row count = 838, but value count = 839

Adding vector validation to the result set loader overflow tests reveals that the problem is in overflow. In TestResultSetLoaderOverflow.testOverflowWithNullables():

a - RepeatedIntVector: Row count = 2952, but value count = 2953
b - RepeatedVarCharVector: Row count = 2952, but value count = 2953
b - RepeatedVarCharVector: Vector has 2953 values, but offset vector labels 32472 values
c - RepeatedIntVector: Row count = 2952, but value count = 2953
d - RepeatedIntVector: Row count = 2952, but value count = 2953

The problem is that EVF incorrectly sets the offset buffer writer index after a rollover.

Issue Links

• links to: GitHub Pull Request #1878