Skip to content

[docs] Fix stale type support, procedure and CDC action references - #9973

Open
thswlsqls wants to merge 1 commit into
apache:masterfrom
thswlsqls:docs/fix-stale-type-support-procedure-and-cdc-references
Open

thswlsqls wants to merge 1 commit into
apache:masterfrom
thswlsqls:docs/fix-stale-type-support-procedure-and-cdc-references

Conversation

@thswlsqls

Copy link
Copy Markdown
Contributor

Purpose

fix #9972

Tests

Docs-only. Each statement checked against source (LanceFileFormat, VortexFileFormat, JsonFileFormat, Expression.NowExpression, BTreeIndexOptions, *SyncDatabaseActionFactory, CompactProcedure, CoreOptions, PaimonMergeIntoResolver, Expire*Procedure, RemoveUnexisting*Procedure). yarn build (as in ci-docs.yml) passes locally.

- fileformat.md: drop MULTISET from the Lance and JSON tables; list the
  types the Lance and Vortex validators reject (BLOB only when inline)
- temporal_functions.html: now() output type is TIMESTAMP(3)
- btree.mdx: document btree-index.high-priority-pool-ratio and
  btree-index.compression-level
- kafka/pulsar_sync_database.html: remove --ignore_incompatible, which
  only mysql_sync_database parses
- compaction.md: document the buckets argument of sys.compact
- mysql-cdc.mdx, mysql_sync_database.html: add --table_conf_by_table
- sql-query.mdx: the option key is source.split.target-size
- sql-write.md: add WHEN NOT MATCHED BY SOURCE to the MERGE INTO syntax
- snapshots.md: older_than argument name; fix the expire_changelogs
  named-argument block
- repair.md: dry_run => true; positional remove_unexisting_manifests
  examples

Generated-by: Claude Code
-- Use named argument
CALL [catalog.]sys.remove_unexisting_files(`table` => 'identifier');
-- Use indexed argument
CALL [catalog.]sys.remove_unexisting_manifests('identifier');

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Minor leftover: the "Arguments" bullet above for remove_unexisting_manifests still says `table`, but the actual argument name is tableId — see @ArgumentHint(name = "tableId") in RemoveUnexistingManifestsProcedure. Since named calls resolve by the hint name, a reader following that bullet with a named call would fail; consider updating the bullet to tableId. The indexed examples changed here look good.

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.

[Bug] Docs contradict the code: file format types, Flink procedures, CDC actions

2 participants