You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@paimon.apache.org by "plusplusjiajia (via GitHub)" <gi...@apache.org> on 2026/07/11 02:56:30 UTC
[PR] [spec] Support parsing Java-serialized Predicate and Transform JSON [paimon-rust]
plusplusjiajia opened a new pull request, #500:
URL: https://github.com/apache/paimon-rust/pull/500
### Purpose
Deserialize Java's Jackson-serialized `Predicate` and `Transform` into Rust
types (`from_java_json` + field-index helpers), for the upcoming query-auth
enforcement (#498 follow-up) and the cross-language protocol direction.
- By-name field resolution, local types authoritative. Java semantics
mirrored at all three layers: wire layout (tests pin
`PredicateJsonSerdeTest`/`TransformJsonSerdeTest` strings verbatim),
constructor rules (`UPPER`/`LOWER` arity 1, `CONCAT_WS` ≥ 2, string-typed
inputs), and evaluator null semantics (null literal → `AlwaysFalse`, `In`
skips nulls, empty `NotIn` → `IS_NOT_NULL`).
- `CAST` validates local-source → target castability; anything unknown
errors, so authorization callers stay fail-closed.
- `DATE`/temporal literals rejected: Java's own deserializer cannot
round-trip them, no wire form to align with yet.
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [spec] Support parsing REST catalog Predicate and Transform JSON [paimon-rust]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi merged PR #500:
URL: https://github.com/apache/paimon-rust/pull/500
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [spec] Support parsing REST catalog Predicate and Transform JSON [paimon-rust]
Posted by "plusplusjiajia (via GitHub)" <gi...@apache.org>.
plusplusjiajia commented on PR #500:
URL: https://github.com/apache/paimon-rust/pull/500#issuecomment-5079383132
> Incompatible with Java legitimate wire format: Java will generate 'LEAF+transform=NULL+function=TRUS/FALSE', which Rust currently explicitly rejects. The result is a failed valid query auth condition (fail closed, not unauthorized, but will erroneously reject the query).
@JingsongLi Thanks for catching this! Fixed — TRUE/FALSE leaves with the NULL transform now parse to AlwaysTrue/AlwaysFalse, with the PredicateJsonSerdeTest wire strings pinned in tests.
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [spec] Support parsing REST catalog Predicate and Transform JSON [paimon-rust]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi commented on PR #500:
URL: https://github.com/apache/paimon-rust/pull/500#issuecomment-4954604065
Incompatible with Java legitimate wire format: Java will generate 'LEAF+transform=NULL+function=TRUS/FALSE', which Rust currently explicitly rejects. The result is a failed valid query auth condition (fail closed, not unauthorized, but will erroneously reject the query).
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org