Skip to content

Produce RunLengthEncodedBlock in ArrayBlockBuilder when all values are null#13973

Closed
lukasz-stec wants to merge 1 commit intotrinodb:masterfrom
starburstdata:ls/040-array-hashnonnullvalue
Closed

Produce RunLengthEncodedBlock in ArrayBlockBuilder when all values are null#13973
lukasz-stec wants to merge 1 commit intotrinodb:masterfrom
starburstdata:ls/040-array-hashnonnullvalue

Conversation

@lukasz-stec
Copy link
Copy Markdown
Member

Description

Add hasNonNullValue support to ArrayBlockBuilder

Is this change a fix, improvement, new feature, refactoring, or other?

improvement

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

only ArrayBlockBuilder so core query engine but change in the spi module

How would you describe this change to a non-technical end user or system administrator?

improve RLE support for Array type

Related issues, pull requests, and links

Similar to #12043 and #12044

Documentation

( X) No documentation is needed.
( ) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

(X ) No release notes entries required.
( ) Release notes entries required with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

Produce RunLengthEncodedBlock in ArrayBlockBuilder
when all values are null
@cla-bot cla-bot bot added the cla-signed label Sep 2, 2022
@lukasz-stec lukasz-stec added performance no-release-notes This pull request does not require release notes entry labels Sep 2, 2022
@lukasz-stec
Copy link
Copy Markdown
Member Author

lukasz-stec commented Sep 2, 2022

moved the changes to #12798

@lukasz-stec lukasz-stec closed this Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed no-release-notes This pull request does not require release notes entry performance

Development

Successfully merging this pull request may close these issues.

1 participant