Skip to content

Conversation

@jaragunde
Copy link
Contributor

We recently changed the getters so they don't create the PPr object if it doesn't exist, but we missed adding one null check for the new situation.

Add a test to exercise all the getters in a case where there isn't a PPr object.

We recently changed the getters so they don't create the PPr object if
it doesn't exist, but we missed adding one null check for the new
situation.

Add a test to exercise all the getters in a case where there isn't a PPr
object.
Copy link
Member

@pjfanning pjfanning left a comment

Choose a reason for hiding this comment

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

Lgtm

@pjfanning pjfanning merged commit 775da74 into apache:trunk Nov 24, 2025
1 check passed
pjfanning pushed a commit that referenced this pull request Nov 24, 2025
We recently changed the getters so they don't create the PPr object if
it doesn't exist, but we missed adding one null check for the new
situation.

Add a test to exercise all the getters in a case where there isn't a PPr
object.
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.

2 participants