Skip to content

[C++] Unsigned dictionary index types should use the full unsigned range #50695

Description

@mkzung

Follow-up to #50475, suggested in review there: an unsigned dictionary index type keeps its signedness, but the width still adapts on the signed thresholds because the dictionary builder reuses the signed AdaptiveIntBuilder internally. A uint8 index widens to uint16 after 128 distinct values instead of 256, so one bit of the range is unused.

Activity

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

Metadata

Metadata

Assignees

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