Skip to content

Ecosystem Integration: pg_tokenizer - #303

Open
messere1 wants to merge 2 commits into
IvorySQL:masterfrom
messere1:codex/issue-1332-pg-tokenizer
Open

Ecosystem Integration: pg_tokenizer#303
messere1 wants to merge 2 commits into
IvorySQL:masterfrom
messere1:codex/issue-1332-pg-tokenizer

Conversation

@messere1

@messere1 messere1 commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add bilingual pg_tokenizer 0.1.1 build, configuration, and usage documentation
  • cover Jieba analysis, English tokenization, and trigger-based automatic token generation
  • document the Oracle-mode identifier-case requirement discovered during validation
  • add the component to the English and Chinese navigation and ecosystem overview

Validation

Built pg_tokenizer 0.1.1 from source without extension source changes against the official IvorySQL 5.4 image (PostgreSQL 18.4) on x86_64 Linux.

  • Rust/pgrx build and installation: passed
  • CREATE EXTENSION pg_tokenizer CASCADE: passed, version 0.1.1
  • Chinese Jieba search analysis: passed
  • English model/filter tokenizer: passed
  • custom-model trigger token generation in PG mode: passed
  • text analysis and tokenization in Oracle mode: passed
  • custom-model trigger in Oracle mode with ivorysql.identifier_case_switch = normal: passed

The documentation explains why the Oracle-mode setting is required and includes the exact workaround for reproducible testing.

Fixes IvorySQL/IvorySQL#1332

Copilot AI lite review requested due to automatic review settings September 4, 2026 03:36

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.

🟡 Changes recommended

There are a few documented formatting consistency issues (including a likely mixed line-ending change in CN nav) that should be corrected before merging to avoid noisy diffs and inconsistent docs style.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Adds bilingual documentation for integrating the pg_tokenizer extension into IvorySQL’s ecosystem docs, including build/install steps, tokenizer/analyzer configuration examples, trigger-based token generation, and an Oracle-mode identifier-case workaround discovered during validation.

Changes:

  • Added new English and Chinese ecosystem component pages for pg_tokenizer (build/install, usage examples, Oracle-mode notes).
  • Linked pg_tokenizer into the English/Chinese ecosystem overview tables.
  • Added pg_tokenizer to the English/Chinese navigation trees.
File summaries
File Description
EN/modules/ROOT/pages/master/ecosystem_components/pg_tokenizer.adoc New English documentation page for building/installing and using pg_tokenizer, including Oracle-mode caveat.
CN/modules/ROOT/pages/master/ecosystem_components/pg_tokenizer.adoc New Chinese documentation page mirroring the English pg_tokenizer guidance.
EN/modules/ROOT/pages/master/ecosystem_components/ecosystem_overview.adoc Adds pg_tokenizer entry to the English ecosystem components overview table.
CN/modules/ROOT/pages/master/ecosystem_components/ecosystem_overview.adoc Adds pg_tokenizer entry to the Chinese ecosystem components overview table.
EN/modules/ROOT/nav.adoc Adds pg_tokenizer page to the English navigation under ecosystem components.
CN/modules/ROOT/nav.adoc Adds pg_tokenizer page to the Chinese navigation and adjusts the surrounding entries.
Review details
  • Files reviewed: 6/6 changed files
  • Comments generated: 4
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread CN/modules/ROOT/nav.adoc Outdated
Comment on lines +78 to +79
*** xref:master/ecosystem_components/set_user.adoc[set_user]
*** xref:master/ecosystem_components/pg_tokenizer.adoc[pg_tokenizer]

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Normalized the Chinese navigation file to a single CRLF newline style.


[source,ini]
----
shared_preload_libraries = 'gb18030_2022,liboracle_parser,ivorysql_ora,pg_tokenizer'

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Added spaces after commas in the preload example.

|*28*| xref:master/ecosystem_components/zhparser_en.adoc[zhparser] | master branch | PostgreSQL extension for full-text search of Chinese language (Mandarin Chinese). It implements a Chinese language parser base on the | Search engine、keyword extraction
|*29*| xref:master/ecosystem_components/pgbackrest.adoc[pgBackRest] | 2.58.0 | pgBackRest is a reliable backup and restore solution for PostgreSQL that seamlessly scales up to the largest databases and workloads | Disaster recovery backup, large database backup, off-site/multi-tier disaster recovery
| *30* | xref:master/ecosystem_components/set_user.adoc[set_user] | REL4_2_0 | PostgreSQL security auditing extension with controlled role switching, supporting allowlists, enforced auditing, and blocking of high-risk operations | Controlled role switching, privilege management, audit logging
| *31* | xref:master/ecosystem_components/pg_tokenizer.adoc[pg_tokenizer] | 0.1.1 | Configurable in-database text analyzers and tokenizers with multilingual processing and trigger-based token generation | Full-text search, RAG preprocessing, multilingual tokenization

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Normalized the new index cell to match the existing table format.


[source,ini]
----
shared_preload_libraries = 'gb18030_2022,liboracle_parser,ivorysql_ora,pg_tokenizer'

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Added spaces after commas in the preload example.

@github-actions
github-actions Bot temporarily deployed to pull request September 4, 2026 03:40 Inactive
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

🚀 IvorySQL-Docs Preview Ready

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

🚀 IvorySQL-Docs Preview Ready

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