Michael Smith has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23714 )
Change subject: IMPALA-13941: Add helper to format file permissions as UNIX-style string ...................................................................... IMPALA-13941: Add helper to format file permissions as UNIX-style string This change introduces a utility method FormatPermissions() that converts mode_t permission bits into a human-readable string (e.g., "drwxrwxrwt"). It correctly handles file type indicators, owner/group/other read-write-execute bits, and special bits such as setuid, setgid, and sticky. This improves log readability and debugging for file metadata-related operations by providing consistent, ls-style permission formatting. Testing: - Added unit tests validating permission string output for: - Regular files, directories, symlinks, sockets - All rwx combinations for user/group/other - setuid, setgid, and sticky bit behavior Change-Id: Ib53dbecd5c202e33b6e3b5cd3a372a77d8b1703a Reviewed-on: http://gerrit.cloudera.org:8080/23714 Reviewed-by: Riza Suminto <[email protected]> Reviewed-by: Michael Smith <[email protected]> Tested-by: Michael Smith <[email protected]> --- M be/src/rpc/authentication-test.cc M be/src/rpc/authentication.cc 2 files changed, 109 insertions(+), 3 deletions(-) Approvals: Riza Suminto: Looks good to me, but someone else must approve Michael Smith: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/23714 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib53dbecd5c202e33b6e3b5cd3a372a77d8b1703a Gerrit-Change-Number: 23714 Gerrit-PatchSet: 5 Gerrit-Owner: Arnab Karmakar <[email protected]> Gerrit-Reviewer: Arnab Karmakar <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]>
