Skip to content

Better code around decompression [2]#27743

Merged
nikitamikhaylov merged 4 commits intoClickHouse:masterfrom
nikitamikhaylov:lz4-checks
Aug 20, 2021
Merged

Better code around decompression [2]#27743
nikitamikhaylov merged 4 commits intoClickHouse:masterfrom
nikitamikhaylov:lz4-checks

Conversation

@nikitamikhaylov
Copy link
Copy Markdown
Member

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Aug 16, 2021
Copy link
Copy Markdown
Member

@KochetovNicolai KochetovNicolai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is changed looks ok to me.
Not sure we didn't miss something

@nikitamikhaylov nikitamikhaylov merged commit ea3f7be into ClickHouse:master Aug 20, 2021
@Algunenano
Copy link
Copy Markdown
Member

Are the changes in performance unrelated?

nikitamikhaylov added a commit that referenced this pull request Nov 8, 2021
Better code around decompression [2]

(cherry picked from commit ea3f7be)
nikitamikhaylov added a commit that referenced this pull request Nov 8, 2021
Better code around decompression [2]

(cherry picked from commit ea3f7be)
nikitamikhaylov added a commit that referenced this pull request Nov 8, 2021
Better code around decompression [2]

(cherry picked from commit ea3f7be)
Enmk pushed a commit to Altinity/ClickHouse that referenced this pull request Mar 23, 2022
Merge pull request ClickHouse#27743 from nikitamikhaylov/lz4-checks

Better code around decompression [2]
nickitat added a commit to nickitat/ClickHouse that referenced this pull request Aug 9, 2022
…checks"

This reverts commit ea3f7be, reversing
changes made to c3c31e2.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants