Skip to content

Show pax relation relfilenode in pg_waldump#1048

Merged
reshke merged 3 commits intoapache:mainfrom
reshke:hcdweui
Apr 23, 2025
Merged

Show pax relation relfilenode in pg_waldump#1048
reshke merged 3 commits intoapache:mainfrom
reshke:hcdweui

Conversation

@reshke
Copy link
Copy Markdown
Contributor

@reshke reshke commented Apr 14, 2025

Currenly, pax_desc prints only pax file numver, not relfilenode

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


@jiaqizho
Copy link
Copy Markdown
Contributor

@gongxun0928

@reshke
Copy link
Copy Markdown
Contributor Author

reshke commented Apr 15, 2025

#1044 (comment)

@reshke
Copy link
Copy Markdown
Contributor Author

reshke commented Apr 15, 2025

I dont understand why CI is not running here.

Comment thread contrib/pax_storage/src/cpp/storage/wal/paxc_desc.c Outdated
@jiaqizho
Copy link
Copy Markdown
Contributor

I dont understand why CI is not running here.

@reshke i rebased u commit and retrigger the CI

reshke and others added 2 commits April 16, 2025 13:07
Currenly, pax_desc prints only pax file numver, not relfilenode
@jiaqizho jiaqizho requested a review from my-ship-it April 17, 2025 02:12
@reshke reshke merged commit 8da79b3 into apache:main Apr 23, 2025
26 checks passed
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.

4 participants