accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From els...@apache.org
Subject [3/3] accumulo git commit: Merge branch '1.7'
Date Fri, 04 Dec 2015 19:52:05 GMT
Merge branch '1.7'


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/afd31edb
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/afd31edb
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/afd31edb

Branch: refs/heads/master
Commit: afd31edb7c5fa33159ab4778b04f9e8727d9ddaa
Parents: 472bdd9 afa475d
Author: Josh Elser <elserj@apache.org>
Authored: Fri Dec 4 14:49:03 2015 -0500
Committer: Josh Elser <elserj@apache.org>
Committed: Fri Dec 4 14:49:03 2015 -0500

----------------------------------------------------------------------
 .../org/apache/accumulo/test/BulkImportSequentialRowsIT.java     | 4 ++++
 1 file changed, 4 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/afd31edb/test/src/main/java/org/apache/accumulo/test/BulkImportSequentialRowsIT.java
----------------------------------------------------------------------
diff --cc test/src/main/java/org/apache/accumulo/test/BulkImportSequentialRowsIT.java
index 0297c14,0000000..1cfd3e6
mode 100644,000000..100644
--- a/test/src/main/java/org/apache/accumulo/test/BulkImportSequentialRowsIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/BulkImportSequentialRowsIT.java
@@@ -1,108 -1,0 +1,112 @@@
 +/*
 + * 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.accumulo.test;
 +
 +import static org.junit.Assert.assertEquals;
 +import static org.junit.Assert.assertTrue;
 +
 +import java.util.TreeSet;
 +
 +import org.apache.accumulo.core.client.admin.TableOperations;
 +import org.apache.accumulo.core.security.Authorizations;
 +import org.apache.accumulo.harness.AccumuloClusterHarness;
 +import org.apache.accumulo.minicluster.impl.MiniAccumuloConfigImpl;
 +import org.apache.hadoop.conf.Configuration;
 +import org.apache.hadoop.fs.FileSystem;
++import org.apache.hadoop.fs.FsShell;
 +import org.apache.hadoop.fs.Path;
 +import org.apache.hadoop.fs.RawLocalFileSystem;
 +import org.apache.hadoop.io.Text;
 +import org.junit.Test;
 +import org.slf4j.Logger;
 +import org.slf4j.LoggerFactory;
 +
 +import com.google.common.collect.Iterables;
 +
 +// ACCUMULO-3967
 +public class BulkImportSequentialRowsIT extends AccumuloClusterHarness {
 +  private static final Logger log = LoggerFactory.getLogger(BulkImportSequentialRowsIT.class);
 +
 +  private static final long NR = 24;
 +  private static final long NV = 42000;
 +
 +  @Override
 +  public int defaultTimeoutSeconds() {
 +    return 60;
 +  }
 +
 +  @Override
 +  public void configureMiniCluster(MiniAccumuloConfigImpl cfg, Configuration hadoopCoreSite)
{
 +    // Need more than one tserver
 +    cfg.setNumTservers(2);
 +
 +    // use raw local file system so walogs sync and flush will work
 +    hadoopCoreSite.set("fs.file.impl", RawLocalFileSystem.class.getName());
 +  }
 +
 +  @Test
 +  public void testBulkImportFailure() throws Exception {
 +    String tableName = getUniqueNames(1)[0];
 +    TableOperations to = getConnector().tableOperations();
 +    to.create(tableName);
 +    FileSystem fs = getFileSystem();
 +    Path rootPath = new Path(fs.makeQualified(getUsableDir()), getClass().getSimpleName());
 +    log.info("Writing to {}", rootPath);
 +    if (fs.exists(rootPath)) {
 +      assertTrue(fs.delete(rootPath, true));
 +    }
 +    assertTrue(fs.mkdirs(rootPath));
 +
 +    Path bulk = new Path(rootPath, "bulk");
 +    log.info("bulk: {}", bulk);
 +    assertTrue(fs.mkdirs(bulk));
 +    Path err = new Path(rootPath, "err");
 +    log.info("err: {}", err);
 +
 +    assertTrue(fs.mkdirs(bulk));
 +    assertTrue(fs.mkdirs(err));
 +
 +    Path rfile = new Path(bulk, "file.rf");
 +
 +    log.info("Generating RFile {}", rfile.toUri().toString());
 +
 +    GenerateSequentialRFile.main(new String[] {"-f", rfile.toUri().toString(), "-nr", Long.toString(NR),
"-nv", Long.toString(NV)});
 +
 +    assertTrue("Expected that " + rfile + " exists, but it does not", fs.exists(rfile));
 +
++    FsShell fsShell = new FsShell(fs.getConf());
++    assertEquals("Failed to chmod " + rootPath, 0, fsShell.run(new String[] {"-chmod", "-R",
"777", rootPath.toString()}));
++
 +    // Add some splits
 +    to.addSplits(tableName, getSplits());
 +
 +    // Then import a single rfile to all the tablets, hoping that we get a failure to import
because of the balancer moving tablets around
 +    // and then we get to verify that the bug is actually fixed.
 +    to.importDirectory(tableName, bulk.toString(), err.toString(), false);
 +
 +    // The bug is that some tablets don't get imported into.
 +    assertEquals(NR * NV, Iterables.size(getConnector().createScanner(tableName, Authorizations.EMPTY)));
 +  }
 +
 +  private TreeSet<Text> getSplits() {
 +    TreeSet<Text> splits = new TreeSet<>();
 +    for (int i = 0; i < NR; i++) {
 +      splits.add(new Text(String.format("%03d", i)));
 +    }
 +    return splits;
 +  }
 +}


Mime
View raw message