Skip to content

Conversation

Tishj
Copy link
Contributor

@Tishj Tishj commented Sep 10, 2024

This PR fixes #13814

Properly set the length for the entry and loop through the data to verify it.

@Tishj Tishj requested a review from Mytherin September 10, 2024 16:06
@Mytherin Mytherin merged commit db9b5b5 into duckdb:main Sep 11, 2024
26 checks passed
@Mytherin
Copy link
Collaborator

Thanks!

github-actions bot pushed a commit to duckdb/duckdb-r that referenced this pull request Sep 25, 2024
[Dev] Minor cleanup to BufferManager and BlockHandle API (duckdb/duckdb#13838)
[C-API Dev] Fix up `"Test DataChunk populate ListVector in C API"` test (duckdb/duckdb#13839)
github-actions bot added a commit to duckdb/duckdb-r that referenced this pull request Sep 25, 2024
[Dev] Minor cleanup to BufferManager and BlockHandle API (duckdb/duckdb#13838)
[C-API Dev] Fix up `"Test DataChunk populate ListVector in C API"` test (duckdb/duckdb#13839)

Co-authored-by: krlmlr <krlmlr@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Test] C-API Populate ListVector test issue
2 participants