[ https://issues.apache.org/jira/browse/HBASE-20838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16541046#comment-16541046 ]
Duo Zhang commented on HBASE-20838: ----------------------------------- +1 on committing to branch-2.1. > Include hbase-server in precommit test if CommonFSUtils is changed > ------------------------------------------------------------------ > > Key: HBASE-20838 > URL: https://issues.apache.org/jira/browse/HBASE-20838 > Project: HBase > Issue Type: Test > Reporter: Yu Li > Assignee: Yu Li > Priority: Major > Fix For: 3.0.0, 2.2.0 > > Attachments: HBASE-20838.patch, HBASE-20838.patch, > HBASE-20838.v2.patch > > > -As per > [discussed|https://issues.apache.org/jira/browse/HBASE-20691?focusedCommentId=16517662&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16517662] > in HBASE-20691, since the setStoragePolicy code is in CommonFSUtils, the > test should be in TestCommonFSUtils- > We don't want to introduce dependency on hadoop-hdfs to hbase-common so > decided to leave the setStoragePolicy related tests in TestCommonFSUtils. > Instead, we will change the personality script to include hbase-server in > unit test if any change made against {{CommonFSUtils}} -- This message was sent by Atlassian JIRA (v7.6.3#76005)