From 812297c576b4d622b06e63b97b0187c221ea4eb4 Mon Sep 17 00:00:00 2001 From: Matt Lord Date: Mon, 12 Aug 2024 22:40:48 -0400 Subject: [PATCH] Create a PR dedicated to improving the table plan logging Signed-off-by: Matt Lord --- .../tabletmanager/vreplication/table_plan_builder.go | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/go/vt/vttablet/tabletmanager/vreplication/table_plan_builder.go b/go/vt/vttablet/tabletmanager/vreplication/table_plan_builder.go index 9d5e92d8f99..2e44fd49e9b 100644 --- a/go/vt/vttablet/tabletmanager/vreplication/table_plan_builder.go +++ b/go/vt/vttablet/tabletmanager/vreplication/table_plan_builder.go @@ -598,12 +598,10 @@ func (tpb *tablePlanBuilder) analyzePK(cols []*ColumnInfo) error { // TODO(shlomi): at some point in the futue we want to make this check stricter. // We could be reading a generated column c1 which in turn selects some other column c2. // We will want t oensure that `c2` is found in select list... - return fmt.Errorf("primary key column %v not found in %s table's select filter or the TableMap event in the GTID", - col, tpb.name) + return fmt.Errorf("primary key column %v not found in select list", col) } if cexpr.operation != opExpr { - return fmt.Errorf("primary key column %v in table %s is not allowed to reference an aggregate expression", - col, tpb.name) + return fmt.Errorf("primary key column %v is not allowed to reference an aggregate expression", col) } cexpr.isPK = true cexpr.dataType = col.DataType