From 46e0e880561150a6362540ca161e7ecf1539ea02 Mon Sep 17 00:00:00 2001 From: Andrew Purtell Date: Wed, 17 Apr 2019 18:54:34 -0700 Subject: [PATCH] Revert "HBASE-21959 - CompactionTool should close the store it uses for compacting files, in order to properly archive compacted files." This reverts commit c1a64aaa1a75abd0a89209c317a3fecd81853fe6. --- .../hbase/regionserver/CompactionTool.java | 2 - .../regionserver/TestCompactionTool.java | 100 ------------------ 2 files changed, 102 deletions(-) delete mode 100644 hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestCompactionTool.java diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CompactionTool.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CompactionTool.java index 0b483d9d30f8..ed6d80e1538c 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CompactionTool.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CompactionTool.java @@ -171,8 +171,6 @@ private void compactStoreFiles(final Path tableDir, final HTableDescriptor htd, } } } while (store.needsCompaction() && !compactOnce); - //We need to close the store properly, to make sure it will archive compacted files - store.close(); } /** diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestCompactionTool.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestCompactionTool.java deleted file mode 100644 index 8e69b4c979df..000000000000 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestCompactionTool.java +++ /dev/null @@ -1,100 +0,0 @@ -/** - * Licensed to the Apache Software Foundation (ASF) under one - * or more contributor license agreements. See the NOTICE file - * distributed with this work for additional information - * regarding copyright ownership. The ASF licenses this file - * to you under the Apache License, Version 2.0 (the - * "License"); you may not use this file except in compliance - * with the License. You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ -package org.apache.hadoop.hbase.regionserver; - -import static org.junit.Assert.assertEquals; - -import java.io.File; -import java.io.FileOutputStream; - -import org.apache.hadoop.conf.Configuration; -import org.apache.hadoop.fs.FileStatus; -import org.apache.hadoop.fs.FileSystem; -import org.apache.hadoop.fs.Path; -import org.apache.hadoop.hbase.HBaseConfiguration; -import org.apache.hadoop.hbase.HBaseTestingUtility; -import org.apache.hadoop.hbase.TableName; -import org.apache.hadoop.hbase.client.Put; -import org.apache.hadoop.hbase.testclassification.MediumTests; -import org.apache.hadoop.hbase.testclassification.RegionServerTests; -import org.apache.hadoop.hbase.util.Bytes; -import org.apache.hadoop.hbase.util.FSUtils; -import org.apache.hadoop.util.ToolRunner; -import org.junit.After; -import org.junit.Before; -import org.junit.Test; -import org.junit.experimental.categories.Category; - -@Category({ MediumTests.class, RegionServerTests.class }) -public class TestCompactionTool { - - private final HBaseTestingUtility testUtil = new HBaseTestingUtility(); - - private HRegion region; - private final static byte[] qualifier = Bytes.toBytes("qf"); - private Path rootDir; - private final TableName tableName = TableName.valueOf(getClass().getSimpleName()); - - @Before - public void setUp() throws Exception { - this.testUtil.startMiniCluster(); - testUtil.createTable(tableName, HBaseTestingUtility.fam1); - String defaultFS = testUtil.getMiniHBaseCluster().getConfiguration().get("fs.defaultFS"); - Configuration config = HBaseConfiguration.create(); - config.set("fs.defaultFS", defaultFS); - String configPath = this.getClass().getClassLoader() - .getResource("hbase-site.xml").getFile(); - config.writeXml(new FileOutputStream(new File(configPath))); - rootDir = testUtil.getDefaultRootDirPath(); - this.region = testUtil.getMiniHBaseCluster().getRegions(tableName).get(0); - } - - @After - public void tearDown() throws Exception { - this.testUtil.shutdownMiniCluster(); - testUtil.cleanupTestDir(); - } - - @Test - public void testCompactedFilesArchived() throws Exception { - for (int i = 0; i < 10; i++) { - this.putAndFlush(i); - } - HStore store = (HStore)region.getStore(HBaseTestingUtility.fam1); - assertEquals(10, store.getStorefilesCount()); - Path tableDir = FSUtils.getTableDir(rootDir, region.getRegionInfo().getTable()); - FileSystem fs = store.getFileSystem(); - String storePath = tableDir + "/" + region.getRegionInfo().getEncodedName() + "/" - + Bytes.toString(HBaseTestingUtility.fam1); - FileStatus[] regionDirFiles = fs.listStatus(new Path(storePath)); - assertEquals(10, regionDirFiles.length); - int result = ToolRunner.run(HBaseConfiguration.create(), new CompactionTool(), - new String[]{"-compactOnce", "-major", storePath}); - assertEquals(0,result); - regionDirFiles = fs.listStatus(new Path(storePath)); - assertEquals(1, regionDirFiles.length); - } - - private void putAndFlush(int key) throws Exception{ - Put put = new Put(Bytes.toBytes(key)); - put.addColumn(HBaseTestingUtility.fam1, qualifier, Bytes.toBytes("val" + key)); - region.put(put); - region.flush(true); - } - -} \ No newline at end of file