tez-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From hit...@apache.org
Subject tez git commit: TEZ-1859. TestGroupedSplits has commented out test: testGzip. (Bikas Saha via hitesh)
Date Fri, 19 Dec 2014 01:32:02 GMT
Repository: tez
Updated Branches:
  refs/heads/master b29c3871c -> a46ecb23d


TEZ-1859. TestGroupedSplits has commented out test: testGzip. (Bikas Saha via hitesh)


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

Branch: refs/heads/master
Commit: a46ecb23df6936c33f56468296a84fc21a89572e
Parents: b29c387
Author: Hitesh Shah <hitesh@apache.org>
Authored: Thu Dec 18 17:31:22 2014 -0800
Committer: Hitesh Shah <hitesh@apache.org>
Committed: Thu Dec 18 17:31:22 2014 -0800

----------------------------------------------------------------------
 CHANGES.txt                                                   | 1 +
 .../org/apache/hadoop/mapred/split/TestGroupedSplits.java     | 7 +++----
 2 files changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tez/blob/a46ecb23/CHANGES.txt
----------------------------------------------------------------------
diff --git a/CHANGES.txt b/CHANGES.txt
index 73ea805..7e1a386 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -6,6 +6,7 @@ Release 0.6.0: Unreleased
 INCOMPATIBLE CHANGES
 
 ALL CHANGES:
+  TEZ-1859. TestGroupedSplits has commented out test: testGzip.
   TEZ-1868. Document how to do Windows builds due to with ACL symlink build changes.
   TEZ-1872. docs/src/site/custom/project-info-report.properties needs license header.
   TEZ-1850. Enable deploy for tez-ui war.

http://git-wip-us.apache.org/repos/asf/tez/blob/a46ecb23/tez-mapreduce/src/test/java/org/apache/hadoop/mapred/split/TestGroupedSplits.java
----------------------------------------------------------------------
diff --git a/tez-mapreduce/src/test/java/org/apache/hadoop/mapred/split/TestGroupedSplits.java
b/tez-mapreduce/src/test/java/org/apache/hadoop/mapred/split/TestGroupedSplits.java
index 0b85748..90fe596 100644
--- a/tez-mapreduce/src/test/java/org/apache/hadoop/mapred/split/TestGroupedSplits.java
+++ b/tez-mapreduce/src/test/java/org/apache/hadoop/mapred/split/TestGroupedSplits.java
@@ -82,7 +82,7 @@ public class TestGroupedSplits {
   // A reporter that does nothing
   private static final Reporter voidReporter = Reporter.NULL;
 
-  //@Test(timeout=10000)
+  @Test(timeout=10000)
   public void testFormat() throws Exception {
     JobConf job = new JobConf(defaultConf);
 
@@ -232,7 +232,7 @@ public class TestGroupedSplits {
   /**
    * Test using the gzip codec for reading
    */
-  //@Test(timeout=10000)
+  @Test(timeout=10000)
   public void testGzip() throws IOException {
     JobConf job = new JobConf(defaultConf);
     CompressionCodec gzip = new GzipCodec();
@@ -256,9 +256,8 @@ public class TestGroupedSplits {
     for (int j=1; j<=3; ++j) {
       format.setDesiredNumberOfSplits(j);
       InputSplit[] splits = format.getSplits(job, 100);
-      if (j==1 || j==3) {
+      if (j==1) {
         // j==1 covers single split corner case
-        // j==3 cases exercises the code where desired == actual
         // and does not do grouping
         Assert.assertEquals("compressed splits == " + j, j, splits.length);
       }


Mime
View raw message