You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by jc...@apache.org on 2012/10/18 23:25:35 UTC

svn commit: r1399863 - in /pig/trunk: CHANGES.txt src/org/apache/pig/Main.java test/org/apache/pig/test/TestPoissonSampleLoader.java

Author: jcoveney
Date: Thu Oct 18 21:25:35 2012
New Revision: 1399863

URL: http://svn.apache.org/viewvc?rev=1399863&view=rev
Log:
PIG-2972: TestPoissonSampleLoader failing on rhel environment (jcoveney)

Modified:
    pig/trunk/CHANGES.txt
    pig/trunk/src/org/apache/pig/Main.java
    pig/trunk/test/org/apache/pig/test/TestPoissonSampleLoader.java

Modified: pig/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/trunk/CHANGES.txt?rev=1399863&r1=1399862&r2=1399863&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Thu Oct 18 21:25:35 2012
@@ -34,6 +34,8 @@ OPTIMIZATIONS
 
 BUG FIXES
 
+PIG-2972: TestPoissonSampleLoader failing on rhel environment (jcoveney)
+
 PIG-2928: Fix e2e test failures in trunk: FilterBoolean_23/24 (cheolsoo via dvryaboy)
 
 Release 0.11.0 (unreleased)
@@ -50,7 +52,7 @@ PIG-2947: Documentation for Rank operato
 
 PIG-2943: DevTests, Refactor Windows checks to use new Util.WINDOWS method for code health (jgordon via dvryaboy)
 
-PIG-2908: Fix unit tests to work with jdk7 (rohini via dvryaboy) 
+PIG-2908: Fix unit tests to work with jdk7 (rohini via dvryaboy)
 
 PIG-2965: RANDOM should allow seed initialization for ease of testing (jcoveney)
 

Modified: pig/trunk/src/org/apache/pig/Main.java
URL: http://svn.apache.org/viewvc/pig/trunk/src/org/apache/pig/Main.java?rev=1399863&r1=1399862&r2=1399863&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/Main.java (original)
+++ pig/trunk/src/org/apache/pig/Main.java Thu Oct 18 21:25:35 2012
@@ -919,11 +919,11 @@ private static String validateLogFile(St
             String scriptFileAbsPath;
             try {
                 scriptFileAbsPath = scriptFile.getCanonicalPath();
+                strippedDownScriptName = getFileFromCanonicalPath(scriptFileAbsPath);
             } catch (IOException ioe) {
                 log.warn("Could not compute canonical path to the script file " + ioe.getMessage());
-                return null;
+                strippedDownScriptName = null;
             }
-            strippedDownScriptName = getFileFromCanonicalPath(scriptFileAbsPath);
         }
     }
 

Modified: pig/trunk/test/org/apache/pig/test/TestPoissonSampleLoader.java
URL: http://svn.apache.org/viewvc/pig/trunk/test/org/apache/pig/test/TestPoissonSampleLoader.java?rev=1399863&r1=1399862&r2=1399863&view=diff
==============================================================================
--- pig/trunk/test/org/apache/pig/test/TestPoissonSampleLoader.java (original)
+++ pig/trunk/test/org/apache/pig/test/TestPoissonSampleLoader.java Thu Oct 18 21:25:35 2012
@@ -102,7 +102,9 @@ public class TestPoissonSampleLoader ext
         }
         return count;
     }
-
+/*
+// This test has been removed because the results are platform dependent.
+// See: PIG-2926
     @Test
     public void testNumSamples() throws IOException {
         //PoissonSampleLoader.DEFAULT_SAMPLE_RATE is 17
@@ -124,7 +126,7 @@ public class TestPoissonSampleLoader ext
         count = testNumSamples("0.0001", "100");
         assertEquals(count, 42);
     }
-
+*/
     /*
      * Test use of LoadFunc with parameters as argument to PoissonSampleLoader
      */