Skip to content

Fix S3 vectors knowledge base validation#68

Merged
gabrielbressan-tfy merged 1 commit into
mainfrom
fix/s3vectors-get-dimension
Jun 10, 2026
Merged

Fix S3 vectors knowledge base validation#68
gabrielbressan-tfy merged 1 commit into
mainfrom
fix/s3vectors-get-dimension

Conversation

@gabrielbressan-tfy

Copy link
Copy Markdown
Collaborator

Summary

  • add get_dimension() to the S3 vectors handler so knowledge base creation validates dimensions via S3 index metadata instead of a broad SELECT
  • treat S3 Vectors ConflictException as an existing index when if_not_exists=True
  • add unit coverage for get_dimension, existing-index retries, and preserving strict SELECT behavior

Tests

  • docker exec dev-mindsdb python -m unittest tests/unit/handlers/test_s3vectors_handler.py

@gabrielbressan-tfy gabrielbressan-tfy merged commit 32f3c2a into main Jun 10, 2026
3 of 5 checks passed
@gabrielbressan-tfy gabrielbressan-tfy deleted the fix/s3vectors-get-dimension branch June 10, 2026 20:01
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.

1 participant