JingsongLi commented on PR #10179: URL: https://github.com/apache/paimon/pull/10179#issuecomment-5831401344
Reviewed the current head (e4174f7) for the REST query-auth use case. This has end-to-end value: a Java-authored rule using these string transforms can otherwise prevent PyPaimon reads. I compared the five implementations with their Java transforms, including Unicode/code-point behavior, UTF-8 bit length, null handling, overlay length defaults, and PAD truncation. I found no blocking correctness issue. Validation: `PYTHONPATH=paimon-python python -m unittest pypaimon.tests.predicate_json_parser_test` (87 passed) and the auth masking/table/validation/widening suites (240 passed); `git diff --check` passed. CI run 36106876365 is green. These tests exercise the parser and auth consumers; a live REST catalog integration remains the production-path check. Please also remove the two `PLACEHOLDER_*` comments before merge. -- 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]
