Skip to content

scan_items/scan_values: accept wrapped stores now that items()/values() are batched #6

Closed

Description

@thorwhalen

After #4's PR, store.items()/values() are correct and batched (BatchGetItem, 100 keys per call) through any dol wrapper. scan_items/scan_values still refuse wrapped stores with a TypeError (existing test test_scan_refuses_wrapped_store); they could delegate to items()/values() for wrapped stores and keep the single-scan path for bare ones. Left as is because it changes a tested public behaviour (relaxing a refusal). Recommendation: do it.

Activity

  1. thorwhalen commented on Oct 13, 2026

    @thorwhalen
    MemberAuthor

    Fixed by #10. Follow-up: #9

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