-
Notifications
You must be signed in to change notification settings - Fork 409
Closed
Labels
affects-5.0affects-5.1affects-5.2affects-5.3affects-5.4This bug affects the 5.4.x(LTS) versions.This bug affects the 5.4.x(LTS) versions.component/computeseverity/criticaltype/bugThe issue is confirmed as a bug.The issue is confirmed as a bug.
Description
Bug Report
Please answer these questions before submitting your issue. Thanks!
1. Minimal reproduce step (Required)
issue 120 112.sql to tiflash
2. What did you expect to see? (Required)
3. What did you see instead (Required)
the minTSO query is blocked thus all queries are blocked.
4. What is your TiFlash version? (Required)
master
the task 9 is hanged, which contains three pipelines.
CreateSet
/ \
build probe
| \
CreateSet recv
/ \
build probe
| |
agg. recv
|
recv
void CreatingSetsBlockInputStream::createAll()
{
if (!created)
{
for (auto & subqueries_for_sets : subqueries_for_sets_list)
{
for (auto & elem : subqueries_for_sets)
{
if (elem.second.join)
elem.second.join->setFinishBuildTable(false);
}
}
Stopwatch watch;
auto thread_manager = newThreadManager();
for (auto & subqueries_for_sets : subqueries_for_sets_list)
{
for (auto & elem : subqueries_for_sets)
{
if (elem.second.source) /// There could be prepared in advance Set/Join - no source is specified for them.
{
if (isCancelledOrThrowIfKilled())
return;
thread_manager->schedule(true, "CreatingSets", [this, &item = elem.second] { createOne(item); });
FAIL_POINT_TRIGGER_EXCEPTION(FailPoints::exception_in_creating_set_input_stream);
}
}
}
thread_manager->wait();
LOG_DEBUG(log, "Creat all tasks of " << mpp_task_id.toString() << " take " << watch.elapsedSeconds() << " sec. ");
.....
we found the last log is missed, so infer the wait() is blocked. The wait() waits the future being ready from the packed_task, which executes the createOne(). the is a bug when the createOne() does not catch all exceptions, the futrue is blocked.
- catch all exceptions for createOne()
- check other places where this bug would occur.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
affects-5.0affects-5.1affects-5.2affects-5.3affects-5.4This bug affects the 5.4.x(LTS) versions.This bug affects the 5.4.x(LTS) versions.component/computeseverity/criticaltype/bugThe issue is confirmed as a bug.The issue is confirmed as a bug.