citusdata / citusdata/citus

Possible performance issue with router planner relation renaming

Open
#682 6 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

UpdateRelationNames() uses 0(n) lookup to find relation restriction for current relation. This could cause performance issues when there are many relations in the query.

This lookup could possibly be replaced with a hash search.

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.