raminqaf opened a new pull request, #29124: URL: https://github.com/apache/flink/pull/29124
## What is the purpose of the change Fixes incorrect documentation for `tryParseJson(allowDuplicateKeys)` introduced in FLINK-40545 (`bf4a71a`). The Javadoc, Python docstring, and `sql_functions.yml` entry were copy-pasted from `parseJson()` and wrongly claimed the `false` branch "throws an error." Since `TRY_PARSE_JSON` catches all parse errors and returns `NULL`, the docs now match that contract. ## Brief change log - `BaseExpressions.java`: fix Javadoc for `tryParseJson(boolean)` - `expression.py`: fix docstring for `try_parse_json` - `sql_functions.yml`: fix description text, trim a trailing-whitespace line ## Verifying this change Docs-only change, no test coverage needed. ## Does this pull request potentially affect one of the following parts: - Dependencies: no - The public API: no (Javadoc/docstring only) - The serializers: no - The runtime per-record code paths: no - Deployment/recovery: no - S3 file system connector: no ## Documentation - Does this pull request introduce a new feature? no --- ##### Was generative AI tooling used to co-author this PR? <!-- If generative AI tooling has been used in the process of authoring this PR, please change the checkbox below to `[X]` and replace the placeholder in the "Generated-by" line with the tool name and version. Otherwise remove the "Generated-by" line. See the ASF Generative Tooling Guidance for details: https://www.apache.org/legal/generative-tooling.html You are responsible for the quality and correctness of every change in this PR regardless of the tooling used. Low-effort AI-generated PRs will be closed. See AGENTS.md for the full guidance. --> - [ ] Yes (please specify the tool below) Generated-by: [Tool Name and Version] -- 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]
