Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove ScalarValue::Dictionary #12488

Commits on Sep 16, 2024

  1. Remove ScalarValue::Dictionary

    `ScalarValue` should be a container for a single nullable logical type
    and should not be concerned by various physical encodings used in
    arrays. It doesn't involve arrays even as part of internal
    representation (like arrow `Scalar` does).
    
    Remove `Dictionary` option from the `ScalarValue`. Note that
    `ScalarValue` doesn't have mapping for RLE arrays, while there are very
    similar type to dictionaries.
    
    Having `ScalarValue` follow complexity of various physical
    representations poses a burden on UDF implementers. They need to handle
    all equivalent scalar values.
    findepi committed Sep 16, 2024
    Configuration menu
    Copy the full SHA
    49a678a View commit details
    Browse the repository at this point in the history