Repository navigation
[read] Complete Java scalar REST authorization casts - #1125
Merged
JingsongLi merged 3 commits intoOct 14, 2026
Merged
Conversation
leaves12138
approved these changes
Oct 14, 2026
leaves12138
left a comment
There was a problem hiding this comment.
Reviewed the latest head c747879. I found no blocking issues. The Java CAST executor matrix, rebinding behavior, nullable mask output handling, and DataFusion authorization projection changes are consistent with the Java semantics. Local verification passed: cargo test --locked -p paimon cast --lib (39), cargo test --locked -p paimon query_auth --lib (80), cargo test --locked -p paimon-datafusion --test query_auth (9), cargo test --locked -p paimon --lib (4037 passed, 6 ignored), cargo test --locked -p paimon --test rest_catalog_test (124), and cargo fmt --all -- --check. The latest GitHub Actions run 38525943981 had all completed jobs passing; the remaining jobs were still queued at review time.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Complete the REST authorization CAST subset that PyPaimon already supports, using Java execution and row getter semantics. The measured 20 × 20 matrix has 212 legal Python/Java executor pairs; Native now covers each of them. Reject the 16 pairs that Python previously admitted but Java has no executor for.
Changes
Transformand Python binding APIs stay unchanged.Verification
fulltext,vortexpass.Representation boundaries
Arrow UTF8 cannot preserve Java BinaryString's invalid byte sequences. Arrow timestamps also have a narrower fixed-unit representation than Java's millis+nanos object. Those results fail explicitly; byte-array Object.toString identity hashes remain unsupported. No NaN or signed-zero vectors were added.
Companion: apache/paimon#10551. Its Native CI continues to build Rust main, so it remains a draft until this Rust change merges.