karmendra8386 commented on PR #14544: URL: https://github.com/apache/gravitino/pull/14544#issuecomment-6608411005
@mchades Both requests are addressed in e1ceb715b: - Added the missing `PROBE_TABLE_LIKE` privilege, mirroring the Java definition (bit 33): the `Privilege.Name` entry, the `ProbeTableLike` privilege class with its type map entry, and the policy allowlist in `AccessControlContent`, where it was the one Java-permitted privilege the Python list couldn't name yet. - Added a deserialization test (`test_access_control_probe_table_like_round_trip` in `test_policy_dto.py`) that decodes an access-control policy containing `SELECT_TABLE` and `PROBE_TABLE_LIKE` and round-trips it, mirroring `TestPolicyDTO.testAccessControlPolicySerDe`. - Full client-python unit suite: 1360 passed, 1022 subtests. Pylint on the changed files: 10.00/10. On the sub-issue: I opened #14593 for this PR's API/DTO scope and updated the PR description to reference it; #7897 stays open for the client operations. One limitation — GitHub returned 404 when I tried to attach #14593 as a formal sub-issue of #7897 (that needs write access on the repo), so #14593 names #7897 as its parent in the body instead. Could you link it as a sub-issue from your side? Fork CI run on this branch: https://github.com/karmendra8386/gravitino/actions/runs/41667597675 -- 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]
