Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

backup: expose descriptor IDs in SHOW BACKUP #67929

Closed
pbardea opened this issue Jul 22, 2021 · 3 comments
Closed

backup: expose descriptor IDs in SHOW BACKUP #67929

pbardea opened this issue Jul 22, 2021 · 3 comments
Assignees
Labels
A-disaster-recovery C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) E-starter Might be suitable for a starter project for new employees or team members. T-disaster-recovery

Comments

@pbardea
Copy link
Contributor

pbardea commented Jul 22, 2021

It would be useful to be able to list descriptor IDs contained inside a backup, especially when combined with an escape hatch as described in #67483.

@pbardea pbardea added C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) A-disaster-recovery E-starter Might be suitable for a starter project for new employees or team members. labels Jul 22, 2021
@pbardea pbardea added this to Triage in Disaster Recovery Backlog via automation Jul 22, 2021
@shermanCRL shermanCRL moved this from Triage to Backup/Restore/Dump in Disaster Recovery Backlog Aug 2, 2021
@dt
Copy link
Member

dt commented Aug 3, 2021

@pbardea you're thinking behind a WITH option, right, e.g. WITH debug_ids or something? And thinking we should add new columns for the id version of everything that has a name column? or just sprintf it into the name (maybe my lazy vote if this is just for debugging, I donno)

@pbardea
Copy link
Contributor Author

pbardea commented Aug 3, 2021

Yep, an option like debug_ids sounds like it'd be a good fit here. I think it's probably worthwhile to add it in it's own column since it'd allow for easier querying on the descriptor names since they'd be well formatted (don't need to worry about where the ID will be in the name).

@dt dt assigned rhu713 Aug 3, 2021
@rhu713
Copy link
Contributor

rhu713 commented Sep 16, 2021

Fixed by #68540

@rhu713 rhu713 closed this as completed Sep 16, 2021
Disaster Recovery Backlog automation moved this from Backup/Restore to Done Sep 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-disaster-recovery C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) E-starter Might be suitable for a starter project for new employees or team members. T-disaster-recovery
Development

No branches or pull requests

3 participants