Skip to content

Bind PartitionSpec to its schema #2530

Open

Description

@dannycjones

When writing logic for rewriting manifest files, it was found to be easy to hand partition specs around with the wrong schema reference: #2367 (comment)

This issue proposes to bind a PartitionSpec to a schema, such as by holding a reference to the Schema. Or, a better approach if there is one.

Ultimately, rough idea:

  • Add schema to PartitionSpec
  • Update SnapshotSummaryCollector's APIs: schema should be inferred from partition spec, not passed in

Activity

  1. dannycjones commented on May 29, 2026

    @dannycjones
    ContributorAuthor

    FYI @CTTY I found this PR removing a type similar to the one we're proposing here: #771

    I will dig a bit more into that PR to understand the reasoning.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions