citusdata / citusdata/citus

Add more failure tests for adaptive executor

Open
#2,943 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C
Stars
12.8k
Forks
794
Avg merge
2d 14h
Merged PRs (30d)
31

Description

With #2942 we've fixed a bug, we should add more test coverage for `StartPlacementExecutionOnSession ()`. And, in general more tests needed for `MULTI_CONNECTION_LOST` case.

@metdos I assigned you because this requires some deeper dive into failure test framework. In this release, or the next, can we allocate some time to add more edge case tests via failure tests? This might require changing some code in failure test framework.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.