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 2010/08/19 02:05:49 UTC
svn commit: r986986 -
/hadoop/pig/trunk/src/org/apache/pig/data/BinInterSedes.java
Author: daijy
Date: Thu Aug 19 00:05:49 2010
New Revision: 986986
URL: http://svn.apache.org/viewvc?rev=986986&view=rev
Log:
Tiny code fix for PIG-1295: Binary comparator for secondary sort
Modified:
hadoop/pig/trunk/src/org/apache/pig/data/BinInterSedes.java
Modified: hadoop/pig/trunk/src/org/apache/pig/data/BinInterSedes.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/data/BinInterSedes.java?rev=986986&r1=986985&r2=986986&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/data/BinInterSedes.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/data/BinInterSedes.java Thu Aug 19 00:05:49 2010
@@ -658,7 +658,7 @@ public class BinInterSedes implements In
// we have a compound tuple key (main_key, secondary_key). Each key has its own sort order, so
// we have to deal with them separately. We delegate it to the first invocation of
// compareBinInterSedesDatum()
- assert (tsz1 == 3); // main_key, secondary_key, value
+ assert (tsz1 == 2); // main_key, secondary_key
result = compareBinInterSedesDatum(bb1, bb2, mAsc);
if (result == 0)
result = compareBinInterSedesDatum(bb1, bb2, mSecondaryAsc);