citusdata / citusdata/citus

Add detailed error messages about router planner failures

Open
#1,164 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

Right now router planner doesn't tell why something failed, we just go on to the distributed planner. That means it's hard to figure out why router failed.

The first step to improve that is to replace all cases where router planner returns NULL with returning an error. Besides being able to print that as a debugging statement, that'll allow us to return good error messages if/when we add a 'router_only' guc (good for migrating apps).

It'd also be good because it improves the error messages in some edge cases (e.g. prepared statments errors via SQL functions)

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.