Skip to content

Comments

HBASE-29899 TestRowStatisticsCompactionObserver fails with NPE#7756

Merged
Apache9 merged 1 commit intoapache:masterfrom
liuxiaocs7:HBASE-29899
Feb 23, 2026
Merged

HBASE-29899 TestRowStatisticsCompactionObserver fails with NPE#7756
Apache9 merged 1 commit intoapache:masterfrom
liuxiaocs7:HBASE-29899

Conversation

@liuxiaocs7
Copy link
Contributor

LOG.info("Record called with isMajor={}, stats={}, fullRegionName={}", stats.isMajor(), stats,
fullRegionName);
lastStats = stats;
lastIsMajor = stats.isMajor();
Copy link
Contributor Author

Choose a reason for hiding this comment

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

L188 is the fix

Comment on lines +50 to 52

@Tag(MediumTests.TAG)
public class TestRowStatisticsCompactionObserver {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

It seems that this unit test hasn't been run before

@Apache9
Copy link
Contributor

Apache9 commented Feb 18, 2026

Seems the tests under row/stats all miss the Category tags?

@liuxiaocs7
Copy link
Contributor Author

liuxiaocs7 commented Feb 20, 2026

Seems the tests under row/stats all miss the Category tags?

Yes, tests under row/stats all miss the Category tags, other migrate use HBASE-29911 (#7776) to track, HBASE-29899 NPE issue we use a separate JIRA here

@Apache9 Apache9 merged commit 6724289 into apache:master Feb 23, 2026
7 checks passed
Apache9 pushed a commit that referenced this pull request Feb 23, 2026
Signed-off-by: Duo Zhang <zhangduo@apache.org>
(cherry picked from commit 6724289)
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