Skip to content

test: harden Arrow LOAD acceptance coverage - #28619

Open
Ariznawlll wants to merge 3 commits into
matrixorigin:mainfrom
Ariznawlll:codex/arrow-feature-full-validation
Open

test: harden Arrow LOAD acceptance coverage#28619
Ariznawlll wants to merge 3 commits into
matrixorigin:mainfrom
Ariznawlll:codex/arrow-feature-full-validation

Conversation

@Ariznawlll

Copy link
Copy Markdown
Collaborator

Summary

  • add real MinIO VersionID controls for Arrow File and Stream readers
  • add transaction, KILL QUERY, and client-disconnect recovery coverage through the MySQL protocol
  • verify reader metrics publish before an outer transaction commits

Verification

  • go test ./pkg/tests/arrowload -count=1 (pass, 132.832s)
  • go test ./pkg/sql/colexec/external -count=1 (pass, 39.213s)

Related

@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M Denotes a PR that changes [100,499] lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants