Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-6891 Hadoop 2 unit test failures
  3. HBASE-8454

TestImportTsv failing due to configuration issues

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Closed
    • Major
    • Resolution: Invalid
    • 0.98.0, 0.94.8, 0.95.1
    • None
    • mapreduce, test
    • None

    Description

      It looks like TestImportTsv can be fixed once HBASE-8453 is applied with this additional delta:

      diff --git a/src/test/java/org/apache/hadoop/hbase/mapreduce/TestImportTsv.java b/src/test/java/org/
      index 77d044b..0da6f0a 100644
      --- a/src/test/java/org/apache/hadoop/hbase/mapreduce/TestImportTsv.java
      +++ b/src/test/java/org/apache/hadoop/hbase/mapreduce/TestImportTsv.java
      @@ -26,6 +26,7 @@ import java.util.ArrayList;
       import org.apache.commons.logging.Log;
       import org.apache.commons.logging.LogFactory;
       import org.apache.hadoop.hbase.*;
      +import org.apache.hadoop.mapred.JobConf;
       import org.apache.hadoop.mapreduce.Job;
       import org.apache.hadoop.fs.FSDataOutputStream;
       import org.apache.hadoop.fs.Path;
      @@ -289,7 +290,10 @@ public class TestImportTsv {
               LOG.info("set the hbaseAdmin");
               ImportTsv.createHbaseAdmin(conf);
             }
      -      Job job = ImportTsv.createSubmittableJob(conf, args);
      +
      +      JobConf jobConf = htu1.getMRCluster().createJobConf();
      +      HBaseConfiguration.merge(jobConf, conf);
      +      Job job = ImportTsv.createSubmittableJob(jobConf, args);
             job.waitForCompletion(false);
             assertTrue(job.isSuccessful());
      

      Tested with Hadoop 2.0.4 + HBase 0.94.7.

      Attachments

        1. 8454.patch
          2 kB
          Andrew Kyle Purtell
        2. 8454-branch-0.94.patch
          1 kB
          Andrew Kyle Purtell

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Unassigned Unassigned
            apurtell Andrew Kyle Purtell
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment