Skip to content

Import gp_url_tools extension from Greenplum (#40)#44

Merged
leborchuk merged 1 commit into
REL_2_STABLEfrom
MoveFixesFromMain2
Jun 30, 2026
Merged

Import gp_url_tools extension from Greenplum (#40)#44
leborchuk merged 1 commit into
REL_2_STABLEfrom
MoveFixesFromMain2

Conversation

@leborchuk

Copy link
Copy Markdown

The initial source is got from https://github.com/open-gpdb/gp_url_tools. Fix decoding of UTF-16 surrogate pairs such as emoji in decode_url(). Make url_tools_schema accessible to all users via GRANT USAGE ON SCHEMA. Add new tests. Make refactoring.

There is the extension description in gpcontrib/gp_url_tools/README.md.

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


The initial source is got from https://github.com/open-gpdb/gp_url_tools.
Fix decoding of UTF-16 surrogate pairs such as emoji in decode_url(). Make
url_tools_schema accessible to all users via GRANT USAGE ON SCHEMA. Add new
tests. Make refactoring.

There is the extension description in gpcontrib/gp_url_tools/README.md.
Copilot AI review requested due to automatic review settings June 29, 2026 14:53

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@leborchuk
leborchuk merged commit 42f1438 into REL_2_STABLE Jun 30, 2026
108 of 112 checks passed
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