Sujal2114 opened a new pull request, #7996: URL: https://github.com/apache/gravitino/pull/7996
What changes were proposed in this pull request? Updated the removeIdFromComment method in StringIdentifier to trim the resulting comment after removing the identifier. This ensures that any trailing spaces left behind are removed, resulting in a clean comment string. Why are the changes needed? Previously, when removing an identifier from a comment, the method sometimes left an extra space at the end. Example: "This is a comment (From Gravitino, DO NOT EDIT: gravitino.v1.uid42)" would become: "This is a comment " // trailing space remains This caused unit test failures and returned inconsistent results. The change fixes this by trimming the comment. Does this PR introduce any user-facing change? Yes. removeIdFromComment will now always return a trimmed comment string, with no leading or trailing whitespace. How was this patch tested? Verified with the unit test testRemoveIdFromCommentTrimsTrailingSpaces that the comment is properly trimmed after identifier removal. Ran the full test suite to confirm no regressions. -- 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]
