Skip to content

[cross-schema] Namespace secondary-schema proto types and enumerate all schemas in catalog metadata - #4445

Draft
arnaud-lacurie wants to merge 2 commits into
apple/arnaud-lacurie/cross-schema/pr3from
apple/arnaud-lacurie/cross-schema/pr4
Draft

[cross-schema] Namespace secondary-schema proto types and enumerate all schemas in catalog metadata#4445
arnaud-lacurie wants to merge 2 commits into
apple/arnaud-lacurie/cross-schema/pr3from
apple/arnaud-lacurie/cross-schema/pr4

Conversation

@arnaud-lacurie

@arnaud-lacurie arnaud-lacurie commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • LogicalOperator prefixes the proto type name of secondary-schema tables with schemaName. (e.g. OTHER.things) to avoid TypeRepository collisions when two schemas define identically-named tables; the FDB record type name (scan key) is preserved unchanged
  • CatalogMetaData.getTables and getColumns now enumerate all schemas in the database when no schema filter is provided, enabling cross-schema metadata inspection

Test plan

  • Joining two schemas that each have a table named things works without TypeRepository collision
  • getTables(database, null, ...) returns tables from all schemas

@arnaud-lacurie arnaud-lacurie added the enhancement New feature or request label Aug 14, 2026
@arnaud-lacurie
arnaud-lacurie force-pushed the apple/arnaud-lacurie/cross-schema/pr4 branch from ff70a9a to ba4ef64 Compare August 14, 2026 16:48
@arnaud-lacurie arnaud-lacurie changed the title Namespace secondary-schema proto types and enumerate all schemas in catalog metadata (cross-schema 4/6) [cross-schema] Namespace secondary-schema proto types and enumerate all schemas in catalog metadata Aug 14, 2026
…atalog metadata

LogicalOperator prefixes secondary-schema type names with the schema name
(e.g. "OTHER_SCHEMA.things") to avoid TypeRepository collisions when two
schemas define identically-named tables, while preserving the original
FDB record type name as the scan key.

CatalogMetaData.getTables/getColumns now enumerates all schemas in the
database when no schema filter is provided, enabling cross-schema metadata
inspection.
@arnaud-lacurie
arnaud-lacurie force-pushed the apple/arnaud-lacurie/cross-schema/pr4 branch from ba4ef64 to eb1c2b3 Compare August 14, 2026 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant