Skip to content

[#9124] Handle negative partNo safely and fix test file typo#9128

Merged
justinmclean merged 3 commits intoapache:mainfrom
HarshMehta112:improvement/9125-fix-revoke-privileges-from-role
Nov 17, 2025
Merged

[#9124] Handle negative partNo safely and fix test file typo#9128
justinmclean merged 3 commits intoapache:mainfrom
HarshMehta112:improvement/9125-fix-revoke-privileges-from-role

Conversation

@HarshMehta112
Copy link
Contributor

Fixed #9124

  • Negative partNo values can now be safely handled.
  • Added unit test cases to verify this behavior.
  • Renamed the test file from TestFulllName to TestFullName to fix a minor typo.

Signed-off-by: Harsh Mehta <harshmehta010102@gmail.com>
Signed-off-by: Harsh Mehta <harshmehta010102@gmail.com>
Signed-off-by: Harsh Mehta <harshmehta010102@gmail.com>
Copy link
Member

@justinmclean justinmclean left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this

@justinmclean justinmclean changed the title Fixed #9124 - Handle negative partNo safely and fix test file typo [#9124] Handle negative partNo safely and fix test file typo Nov 17, 2025
@justinmclean justinmclean merged commit af79aef into apache:main Nov 17, 2025
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Improvement] Fix hasNamePart in FullName.java to check for negative indexes

2 participants