Skip to content

fix(test): adapt procedural language test for Cloudberry behavior#24

Merged
gfphoenix78 merged 1 commit intoapache:sync-with-upstreamfrom
robertmu:create-statemnt
Aug 28, 2025
Merged

fix(test): adapt procedural language test for Cloudberry behavior#24
gfphoenix78 merged 1 commit intoapache:sync-with-upstreamfrom
robertmu:create-statemnt

Conversation

@robertmu
Copy link
Copy Markdown
Contributor

The integration test for procedural languages was failing on Cloudberry due to differences in dependency handling compared to GPDB 7.

In Cloudberry (based on PostgreSQL 14), extension-created languages are marked as "core extension members" and are correctly filtered out by the existing GetProceduralLanguages() function to avoid backup duplication.

This commit updates the test to:

  1. Create the extension first in Cloudberry to make handlers available
  2. Use appropriate cleanup strategy (DROP EXTENSION vs DROP LANGUAGE)
  3. Expect length 0 for Cloudberry (filtered) vs length 1 for GPDB

The changes ensure test compatibility while confirming that the backup logic correctly handles different PostgreSQL dependency models.

Fixes: Integration test failure in predata_functions_create_test.go

The integration test for procedural languages was failing on Cloudberry
due to differences in dependency handling compared to GPDB 7.

In Cloudberry (based on PostgreSQL 14), extension-created languages are
marked as "core extension members" and are correctly filtered out by the
existing GetProceduralLanguages() function to avoid backup duplication.

This commit updates the test to:
1. Create the extension first in Cloudberry to make handlers available
2. Use appropriate cleanup strategy (DROP EXTENSION vs DROP LANGUAGE)
3. Expect length 0 for Cloudberry (filtered) vs length 1 for GPDB

The changes ensure test compatibility while confirming that the backup
logic correctly handles different PostgreSQL dependency models.

Fixes: Integration test failure in predata_functions_create_test.go
@gfphoenix78 gfphoenix78 merged commit a0ff77b into apache:sync-with-upstream Aug 28, 2025
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