Skip to content

Headers for CSVWithNames in StorageFile#12197

Merged
tavplubix merged 2 commits intoClickHouse:masterfrom
nikitamikhaylov:csv-file-bugfix
Jul 8, 2020
Merged

Headers for CSVWithNames in StorageFile#12197
tavplubix merged 2 commits intoClickHouse:masterfrom
nikitamikhaylov:csv-file-bugfix

Conversation

@nikitamikhaylov
Copy link
Member

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

Changelog category (leave one):

  • Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fixed the behaviour when during multiple sequential inserts in StorageFile header for some special types was written more than once. This fixed #6155.

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Jul 7, 2020
@tavplubix tavplubix self-assigned this Jul 7, 2020
@tavplubix tavplubix merged commit 33c3545 into ClickHouse:master Jul 8, 2020
alesapin pushed a commit that referenced this pull request Jul 10, 2020
Headers for CSVWithNames in StorageFile

(cherry picked from commit 33c3545)
alesapin pushed a commit that referenced this pull request Jul 10, 2020
Headers for CSVWithNames in StorageFile

(cherry picked from commit 33c3545)
nikitamikhaylov added a commit that referenced this pull request Jul 14, 2020
…12353)

* done

* fix test

Co-authored-by: Nikita Mikhaylov <[email protected]>
Co-authored-by: tavplubix <[email protected]>
@azat azat mentioned this pull request Aug 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Pull request with bugfix, not backported by default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Engine File(CSVWithNames) inserts more headers than needed

4 participants