[ https://issues.apache.org/jira/browse/HBASE-21175?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16665347#comment-16665347 ]
Artem Ervits commented on HBASE-21175: -------------------------------------- [~yuzhih...@gmail.com] please review v.05 addressing any newly-introduced and existing checkstyle warnings. > Partially initialized SnapshotHFileCleaner leads to NPE during > TestHFileArchiving > --------------------------------------------------------------------------------- > > Key: HBASE-21175 > URL: https://issues.apache.org/jira/browse/HBASE-21175 > Project: HBase > Issue Type: Test > Reporter: Ted Yu > Assignee: Artem Ervits > Priority: Minor > Labels: snapshot > Attachments: HBASE-21175.v01.patch, HBASE-21175.v04.patch, > HBASE-21175.v05.patch > > > TestHFileArchiving#testCleaningRace creates HFileCleaner instance within the > test. > When SnapshotHFileCleaner.init() is called, there is no master parameter > passed in {{params}}. > When the chore runs the cleaner during the test, NPE comes out of this line > in getDeletableFiles(): > {code} > return cache.getUnreferencedFiles(files, master.getSnapshotManager()); > {code} > since master is null. > We should either check for the null master or, pass master instance properly > when constructing the cleaner instance. -- This message was sent by Atlassian JIRA (v7.6.3#76005)