Apache9 commented on code in PR #5245:
URL: https://github.com/apache/hbase/pull/5245#discussion_r1199003624
##########
hbase-compression/hbase-compression-snappy/src/test/java/org/apache/hadoop/hbase/io/compress/xerial/TestHFileCompressionSnappy.java:
##########
@@ -37,10 +39,15 @@ public class TestHFileCompressionSnappy extends
HFileTestBase {
public static final HBaseClassTestRule CLASS_RULE =
HBaseClassTestRule.forClass(TestHFileCompressionSnappy.class);
+ private static final Logger LOG =
LoggerFactory.getLogger(TestHFileCompressionSnappy.class);
private static Configuration conf;
@BeforeClass
public static void setUpBeforeClass() throws Exception {
+ if (!SnappyCodec.isLoaded()) {
Review Comment:
We could use Assume.assumeTrue here to skip the test.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
