zhangshenghang opened a new pull request, #11197:
URL: https://github.com/apache/seatunnel/pull/11197

   ## What documentation issues were found
   
   Several Transform documentation pages were inconsistent with the current 
implementation:
   
   - `FieldEncrypt` documented the wrong mode/default details in places, missed 
the `max_field_length` option, and the Chinese table listed `AES_CBC` although 
the implementation defaults to `AES_GCM`.
   - `JsonPath` documented `path`, `dest_field`, and `dest_type` as string-only 
even though the implementation supports string or array values for batch 
extraction.
   - `Split` had a copied description for `separator` in English and its 
example used `second_name` while the expected output table used `last_name`.
   - `Sql` missed the `engine` option, which defaults to `ZETA` and supports 
`ZETA`/`INTERNAL`.
   - The Chinese `DataValidator` option table used `error_handle_way`, while 
the real option name is `row_error_handle_way`.
   
   ## What was changed
   
   - Updated option tables to match the current config names, defaults, and 
supported value shapes.
   - Added missing `FieldEncrypt.max_field_length` and `Sql.engine` 
documentation.
   - Clarified JsonPath batch extraction array requirements.
   - Corrected Split examples so the configured output fields match the result 
tables.
   - Fixed the Chinese DataValidator option name.
   
   ## Areas/files updated
   
   - `docs/en/transforms/encrypt.md`
   - `docs/zh/transforms/encrypt.md`
   - `docs/en/transforms/jsonpath.md`
   - `docs/zh/transforms/jsonpath.md`
   - `docs/en/transforms/split.md`
   - `docs/zh/transforms/split.md`
   - `docs/en/transforms/sql.md`
   - `docs/zh/transforms/sql.md`
   - `docs/zh/transforms/data-validator.md`
   
   ## English and Chinese docs check
   
   Both English and Chinese Transform documentation were checked. When the same 
issue existed in both languages, both pages were updated together. The 
DataValidator option-name issue was only present in the Chinese page.
   
   ## Duplicate PR check
   
   Before opening this PR, I checked PRs created in the last 7 days with `gh 
search prs --repo apache/seatunnel --created '>=2026-06-20'`. I also inspected 
the files changed by the recent documentation-related PRs, including #11176, 
#11175, #11172, #11164, #11152, #11149, #11148, #11147, #11146, and #11145. 
Those PRs covered JDBC docs, connector links, S3File, Pulsar docs, 
engine/architecture pages, onboarding docs, and README updates. None covered 
these Transform configuration pages.
   
   ## Verification
   
   - `./mvnw spotless:apply`
   - `./mvnw -q -DskipTests verify`
   - `git diff --check`
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to