cloudberry icon indicating copy to clipboard operation
cloudberry copied to clipboard

chore: change the prompt of pax extension in configure help

Open gongxun0928 opened this issue 8 months ago • 2 comments

The --enable-pax configuration option defaults to 'yes'. so in the default help information, we need to display the '--disable-pax' option

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


gongxun0928 avatar Apr 24 '25 03:04 gongxun0928

IMO, as a contrib extension, that would be better to disable it by default.

However, we already use the --enable-pax option in some files, could you help update them together, including:

  • https://github.com/apache/cloudberry/blob/main/.github/workflows/coverity.yml
  • https://github.com/apache/cloudberry/blob/main/.github/workflows/sonarqube.yml

Thanks.

Moreover, welcome to update your commit title by following the template.

tuhaihe avatar Apr 24 '25 03:04 tuhaihe

Hi, I started a discussion on changing PAX plugin to be disabled by default: https://lists.apache.org/thread/q1n7hztzzydk1owhn73tc1b4chyhzx85

tuhaihe avatar Apr 25 '25 08:04 tuhaihe

As discussed, PAX will be disabled by default. So close it.

The new PAX-disabled PR #https://github.com/apache/cloudberry/pull/1081

tuhaihe avatar May 16 '25 10:05 tuhaihe