Skip to content

Fix assertion failure when running \dit+.#1458

Merged
tuhaihe merged 1 commit intoapache:mainfrom
zhangyue1818:psql
Nov 28, 2025
Merged

Fix assertion failure when running \dit+.#1458
tuhaihe merged 1 commit intoapache:mainfrom
zhangyue1818:psql

Conversation

@zhangyue1818
Copy link
Contributor

The translate_columns array had 9 elements but the query returned 10 columns when both tables and indexes were shown. Fixed by:

  1. Expanding translate_columns array to 10 elements
  2. Properly tracking cols_so_far when Storage column is added

Fixes #ISSUE_Number

What does this PR do?

Type of Change

  • Bug fix (non-breaking change)
  • New feature (non-breaking change)
  • Breaking change (fix or feature with breaking changes)
  • Documentation update

Breaking Changes

Test Plan

  • Unit tests added/updated
  • Integration tests added/updated
  • Passed make installcheck
  • Passed make -C src/test installcheck-cbdb-parallel

Impact

Performance:

User-facing changes:

Dependencies:

Checklist

Additional Context

CI Skip Instructions


The translate_columns array had 9 elements but the query returned 10 columns
when both tables and indexes were shown. Fixed by:
1. Expanding translate_columns array to 10 elements
2. Properly tracking cols_so_far when Storage column is added
@tuhaihe tuhaihe merged commit 2644034 into apache:main Nov 28, 2025
41 checks passed
@my-ship-it
Copy link
Contributor

fix #1455

@zhangyue1818 zhangyue1818 deleted the psql branch December 4, 2025 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants