Skip to content

[connector] replace Trino DictionaryBlock with our own implementation #86

@bianhq

Description

@bianhq

Currently, Trino DictionaryBlock stores null value in the dictionary. This is weird and is not compatible with the way that Pixels DictionaryColumnVector handles null values.
In Issue #84, we fix this problem by appending a null value to the dictionary block if there are null values in the column vector. However, this is inefficient as it leads to memory copies and array iterations. We should add our own DictionaryBlock implementation.

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions