pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Tom Lane <tgl@sss.pgh.pa.us>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Include error location in errors from ComputeIndexAttrs().
Date: Sun, 04 Jan 2026 19:16:33 +0000
Message-ID: <E1vcTaa-004UdF-1i@gemulon.postgresql.org> (raw)

Include error location in errors from ComputeIndexAttrs().

Make use of IndexElem's new location field to localize these
errors better.

Author: jian he <jian.universality@gmail.com>
Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/CACJufxH3OgXF1hrzGAaWyNtye2jHEmk9JbtrtGv-KJK6tsGo5w@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/ba75f717526cbaa9000b546aac456e43d03aaf53

Modified Files
--------------
src/backend/bootstrap/bootparse.y                  |  6 ++-
src/backend/commands/indexcmds.c                   | 61 +++++++++++++++-------
src/backend/commands/tablecmds.c                   |  9 ++--
src/backend/tcop/utility.c                         |  3 +-
src/include/commands/defrem.h                      |  3 +-
src/test/regress/expected/alter_table.out          |  4 ++
src/test/regress/expected/collate.icu.utf8.out     |  2 +
src/test/regress/expected/collate.linux.utf8.out   |  2 +
src/test/regress/expected/collate.out              |  2 +
.../regress/expected/collate.windows.win1252.out   |  2 +
src/test/regress/expected/sqljson_queryfuncs.out   | 56 ++++++++++++++++++++
11 files changed, 123 insertions(+), 27 deletions(-)



Message-ID: <E1vcTaa-004UdF-1i@gemulon.postgresql.org>
Permalink:  ../E1vcTaa-004UdF-1i@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1vcTaa-004UdF-1i@gemulon.postgresql.org

 · 

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: tgl@sss.pgh.pa.us, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Include error location in errors from ComputeIndexAttrs().
  In-Reply-To: <E1vcTaa-004UdF-1i@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox