Skip to content

Build tics docker image - #6

Merged
zanmato1984 merged 2 commits into
raftfrom
raft-docker-tics
Mar 5, 2019
Merged

Build tics docker image#6
zanmato1984 merged 2 commits into
raftfrom
raft-docker-tics

Conversation

@zanmato1984

Copy link
Copy Markdown
Contributor

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

@zanmato1984
zanmato1984 merged commit 6250291 into raft Mar 5, 2019
@zanmato1984
zanmato1984 deleted the raft-docker-tics branch March 5, 2019 11:08
windtalker added a commit to windtalker/tiflash that referenced this pull request Mar 6, 2023
* fix hang issue and some potential crash issue

Signed-off-by: xufei <xufei@pingcap.com>

* fix some bugs

Signed-off-by: xufei <xufeixw@mail.ustc.edu.cn>

* fix build

Signed-off-by: xufei <xufeixw@mail.ustc.edu.cn>

---------

Signed-off-by: xufei <xufei@pingcap.com>
Signed-off-by: xufei <xufeixw@mail.ustc.edu.cn>
zanmato1984 added a commit that referenced this pull request Apr 7, 2026
Role: Coder-R138: add TiFlash donor CAST host-v2 parity entrypoint
premal added a commit to 6si/tiflash that referenced this pull request Jun 25, 2026
…esponseWriter

Bug pingcap#5: DMFileReader::readWithFilter calls insertSelectiveFrom/insertRangeFrom
on ColumnDictionary columns; ColumnString::insertSelectiveRangeFrom does an
unchecked static_cast<ColumnString> which returns garbage sizes and causes
128 TiB allocation.

Bug pingcap#6: StreamingDAGResponseWriter (root MPP task writer) did not call
materializeBlocks() before encoding, leaving ColumnDictionary columns for
memory accounting (block.allocatedBytes()) which could interact with memory
tracking code paths and cause 8 EiB allocation.

Both fixes add convertToFullColumnIfDictionary() in the affected paths.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant