Skip to content

feat: tweak internal representation of lookups #969

Description

@DavePearce

I'm not happy with the internal representation of LookupConstraint because it (confusingly) embeds the selectors in the source / target columns. My feeling is that this is likely to cause subtle bugs where this fact is forgotten. Instead, it would be better to make these selectors explicit.

A good solution here would be to introduce a LookupVector or similar, meaning a lookup goes from an array of vectors to an array of vectors, etc.

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