Skip to content

Implement CatalogClient class for catalog operations#27

Merged
rustyconover merged 2 commits into
mainfrom
feature/catalog-client
Jan 6, 2026
Merged

Implement CatalogClient class for catalog operations#27
rustyconover merged 2 commits into
mainfrom
feature/catalog-client

Conversation

@rustyconover

Copy link
Copy Markdown
Contributor

Summary

  • Add CatalogClient class for invoking CatalogInterface methods on VGI workers
  • Each method call spawns new worker (short-lived pattern)
  • Simplified protocol: invoke → stream (no bind/init phases)
  • All CatalogInterface methods mirrored with proper types

Core methods:

  • catalogs() - List available catalogs
  • catalog_attach/detach - Catalog lifecycle
  • schemas(), schema_get/create/drop - Schema operations
  • table_get/create/drop, table_scan_function_get - Table operations
  • view_get/create/drop - View operations
  • Transaction methods: begin/commit/rollback

Dependencies

This PR includes changes from:

Test plan

  • All existing tests pass (661 tests)
  • mypy type checks pass
  • ruff linting passes

🤖 Generated with Claude Code

rustyconover and others added 2 commits January 5, 2026 19:48
Add CatalogClient for invoking CatalogInterface methods on VGI workers:
- Each method call spawns new worker (short-lived pattern)
- Simplified protocol: invoke → stream (no bind/init phases)
- All CatalogInterface methods mirrored with proper types

Core methods:
- catalogs() - List available catalogs
- catalog_attach/detach - Catalog lifecycle
- schemas(), schema_get/create/drop - Schema operations
- table_get/create/drop, table_scan_function_get - Table operations
- view_get/create/drop - View operations
- Transaction methods: begin/commit/rollback

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@rustyconover rustyconover merged commit bfd7d84 into main Jan 6, 2026
0 of 16 checks passed
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