You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by da...@apache.org on 2014/03/17 05:22:56 UTC

svn commit: r1578224 - in /pig/trunk: CHANGES.txt test/perf/pigmix/bin/runpigmix.pl

Author: daijy
Date: Mon Mar 17 04:22:56 2014
New Revision: 1578224

URL: http://svn.apache.org/r1578224
Log:
PIG-3673: Divide by zero error in runpigmix.pl script

Modified:
    pig/trunk/CHANGES.txt
    pig/trunk/test/perf/pigmix/bin/runpigmix.pl

Modified: pig/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/trunk/CHANGES.txt?rev=1578224&r1=1578223&r2=1578224&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Mon Mar 17 04:22:56 2014
@@ -99,6 +99,8 @@ OPTIMIZATIONS
  
 BUG FIXES
 
+PIG-3673: Divide by zero error in runpigmix.pl script (suhassatish via daijy)
+
 PIG-3805: ToString(datetime [, format string]) doesn't work without the second argument (jennythompson via daijy)
 
 PIG-3809: AddForEach optimization doesn't set the alias of the added foreach (cheolsoo)

Modified: pig/trunk/test/perf/pigmix/bin/runpigmix.pl
URL: http://svn.apache.org/viewvc/pig/trunk/test/perf/pigmix/bin/runpigmix.pl?rev=1578224&r1=1578223&r2=1578224&view=diff
==============================================================================
--- pig/trunk/test/perf/pigmix/bin/runpigmix.pl (original)
+++ pig/trunk/test/perf/pigmix/bin/runpigmix.pl Mon Mar 17 04:22:56 2014
@@ -80,7 +80,10 @@ for(my $i = 1; $i <= 17; $i++) {
         $mr_times = int($mr_times + 0.5);
         $total_mr_times = $total_mr_times + $mr_times;
 
-        my $multiplier = $pig_times/$mr_times;
+
+	my $multiplier=0;
+	if ($mr_times!=0) 
+	    $multiplier = $pig_times/$mr_times;
         print "PigMix_$i pig run time: $pig_times, java run time: $mr_times, multiplier: $multiplier\n";
     }
 }