Skip to content

Conversation

@Mulily0513
Copy link

@Mulily0513 Mulily0513 commented Aug 12, 2025

Fixes #ISSUE_Number

What does this PR do?

The most of feature is done by upsteam, but hot DR qd can not
organzie cdbcomponent by gp_segment_configuration rel. Define
a boolean GUC name as hot_dr, if the hot_dr enable, read
cluster infor from segconf file.

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


@weinan003
Copy link
Contributor

帮忙调查一下isolation2中installcheck-hot-standby测试集是否已经在内核CI中运行了,如果没有的话,我们需要考虑将这部分测试集在内核或CBDR项目的CI中启动

@Mulily0513
Copy link
Author

帮忙调查一下isolation2中installcheck-hot-standby测试集是否已经在内核CI中运行了,如果没有的话,我们需要考虑将这部分测试集在内核或CBDR项目的CI中启动

测试集没有在内核ci中运行,这部分需要在github ci中添加

@Mulily0513 Mulily0513 changed the title Add guc gp_segment_configuration_file for hot standby Add guc hot_dr for hot standby Aug 18, 2025
Copy link
Contributor

@weinan003 weinan003 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@my-ship-it my-ship-it force-pushed the archive_restore branch 2 times, most recently from 8dce9d3 to 05d9aa6 Compare August 18, 2025 13:09
The most of feature is done by upsteam, but hot DR qd can not
organzie cdbcomponent by `gp_segment_configuration` rel. Define
a boolean GUC name as `hot_dr`, if the `hot_dr` enable, read
cluster infor from segconf file.
@weinan003 weinan003 merged commit 92e3374 into apache:main Aug 19, 2025
27 checks passed
@Mulily0513 Mulily0513 deleted the archive_restore branch September 1, 2025 02:25
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