Skip to content

Conversation

@KiteSoar
Copy link

@KiteSoar KiteSoar commented Feb 2, 2026

Purpose

Linked issue: close #2521

Brief change log

  • Enhance TableLookup.createLookuper() to intelligently route lookup requests:
    • Create PrimaryKeyLookuper when lookup columns match primary key
    • Create PrefixKeyLookuper when lookup columns are valid prefix key
    • Fail-fast with clear error message for invalid lookup columns

Tests

  • Add TableLookupTest with 10 test cases covering:
    • Primary key lookup (with/without lookupBy, partitioned/non-partitioned)
    • Prefix key lookup (single/multiple columns, partitioned/non-partitioned)
    • Invalid inputs (out-of-order, empty, non-primary/non-prefix columns)
    • Log table rejection

API and Format

None

Documentation

None

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.

lookupBy should align with the method name to support do primary key lookup

1 participant