sundapeng commented on PR #8785: URL: https://github.com/apache/paimon/pull/8785#issuecomment-5036274334
Good point, thanks — reworked exactly that way. The request now carries the JSON serialization of a Paimon `Predicate` tree (the same carrier the auth row filters use with `TableQueryAuthResult#remapPredicate`), instead of a separate expression grammar. That removes the whole translation layer: the server deserializes, re-anchors fields by name against its partition schema, and evaluates the same tree the scan evaluates locally. One note on the contract: unlike the auth row filter (a mandate the engine must fully enforce), this predicate is a best-effort pushdown hint — a tree the server cannot restore (e.g. version skew) or re-anchor counts as always-true, so the response is a superset and the client keeps filtering locally. That also means the wire format can evolve with `Predicate` serialization without version negotiation. -- 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]
