Skip to content

Build: make diskquota installation opt-in#1664

Open
tuhaihe wants to merge 1 commit intoapache:mainfrom
tuhaihe:update-diskquota-install-way
Open

Build: make diskquota installation opt-in#1664
tuhaihe wants to merge 1 commit intoapache:mainfrom
tuhaihe:update-diskquota-install-way

Conversation

@tuhaihe
Copy link
Copy Markdown
Member

@tuhaihe tuhaihe commented Apr 7, 2026

diskquota is currently built and installed by default from gpcontrib, which makes it available in all standard builds without an explicit decision to enable it.

Change diskquota to an opt-in installation model by adding a dedicated --with-diskquota configure option that defaults to no. Only build, install, and run installcheck for diskquota when that option is explicitly enabled.

This makes diskquota adoption an explicit choice and avoids shipping the extension by default.

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


@tuhaihe tuhaihe marked this pull request as draft April 7, 2026 03:58
diskquota is currently built and installed by default from gpcontrib,
which makes it available in all standard builds without an explicit
decision to enable it.

Change diskquota to an opt-in installation model by adding a dedicated
`--with-diskquota` configure option that defaults to `no`. Only build,
install, and run installcheck for diskquota when that option is
explicitly enabled.

This makes diskquota adoption an explicit choice and avoids shipping the
extension by default.
@tuhaihe tuhaihe force-pushed the update-diskquota-install-way branch from 845cd5e to ba02370 Compare April 7, 2026 07:05
@tuhaihe tuhaihe marked this pull request as ready for review April 7, 2026 07:06
@tuhaihe tuhaihe requested a review from leborchuk April 7, 2026 10:33
Copy link
Copy Markdown
Contributor

@leborchuk leborchuk left a comment

Choose a reason for hiding this comment

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

LGTM, lets not pack diskquota as a default extension as we did for other extensions

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.

2 participants