pingcap/tidb

Extend 'Table has no partition for value ...' error message to include the value

Open

#60.629 geöffnet am 17. Apr. 2025

Auf GitHub ansehen
 (9 Kommentare) (0 Reaktionen) (2 zugewiesene Personen)Go (40.090 Stars) (6.186 Forks)batch import
good first issuetype/enhancement

Beschreibung

Enhancement

Test:

tidb> create table t (id varchar(255)) partition by list columns (id) (partition p0 values in ('ok'));
Query OK, 0 rows affected (0.03 sec)

tidb> insert into t values ('ok'),('notok');
ERROR 1526 (HY000): Table has no partition for value from column_list

It would be more beneficial if the error was:

ERROR 1526 (HY000): Table has no partition for value 'notok'

instead

Contributor Guide

Tech Stack
gosql
Domain
databasebackenddeveloper experience
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
blocked
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Go basicsTiDB partitioning
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65
Research-Richtung
Locate the code that generates the error message 'Table has no partition for value from column list' in TiDB's partition handling (likely in 'table/partition.go' or similar). Modify the error string to include the actual column value that is not found in any partition. Ensure the value is properly quoted to avoid SQL injection in the error output. Check the existing tests for partition error messages to add a test case covering this new error format.