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

svn commit: r1582376 - in /pig/trunk: CHANGES.txt src/org/apache/pig/newplan/logical/relational/LogicalPlan.java

Author: cheolsoo
Date: Thu Mar 27 16:16:10 2014
New Revision: 1582376

URL: http://svn.apache.org/r1582376
Log:
PIG-3836: Pig signature has has guava version dependency (amatsukawa via cheolsoo)

Modified:
    pig/trunk/CHANGES.txt
    pig/trunk/src/org/apache/pig/newplan/logical/relational/LogicalPlan.java

Modified: pig/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/trunk/CHANGES.txt?rev=1582376&r1=1582375&r2=1582376&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Thu Mar 27 16:16:10 2014
@@ -103,6 +103,8 @@ OPTIMIZATIONS
  
 BUG FIXES
 
+PIG-3836: Pig signature has has guava version dependency (amatsukawa via cheolsoo)
+
 PIG-3807: Pig creates wrong schema after dereferencing nested tuple fields with sorts (daijy)
 
 PIG-3813: Rank column is assigned different uids everytime when schema is reset (cheolsoo)

Modified: pig/trunk/src/org/apache/pig/newplan/logical/relational/LogicalPlan.java
URL: http://svn.apache.org/viewvc/pig/trunk/src/org/apache/pig/newplan/logical/relational/LogicalPlan.java?rev=1582376&r1=1582375&r2=1582376&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/newplan/logical/relational/LogicalPlan.java (original)
+++ pig/trunk/src/org/apache/pig/newplan/logical/relational/LogicalPlan.java Thu Mar 27 16:16:10 2014
@@ -81,12 +81,12 @@ public class LogicalPlan extends BaseOpe
             ps.println("<logicalPlan>XML Not Supported</logicalPlan>");
             return;
         }
-        
+
         ps.println("#-----------------------------------------------");
         ps.println("# New Logical Plan:");
         ps.println("#-----------------------------------------------");
 
-        if (this.size() == 0) { 
+        if (this.size() == 0) {
             ps.println("Logical plan is empty.");
         } else if (format.equals("dot")) {
             DotLOPrinter lpp = new DotLOPrinter(this, ps);
@@ -126,8 +126,8 @@ public class LogicalPlan extends BaseOpe
      */
     public String getSignature() throws FrontendException {
 
-        // Use a streaming hash function. goodFastHash(32) is murmur3 32 bits
-        HashFunction hf = Hashing.goodFastHash(32);
+        // Use a streaming hash function. We use a murmur_32 function with a constant seed, 0.
+        HashFunction hf = Hashing.murmur3_32(0);
         HashOutputStream hos = new HashOutputStream(hf);
         PrintStream ps = new PrintStream(hos);