You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by dv...@apache.org on 2012/09/03 00:53:00 UTC
svn commit: r1380085 - in /pig/trunk: CHANGES.txt
src/org/apache/pig/newplan/logical/relational/LOSort.java
Author: dvryaboy
Date: Sun Sep 2 22:52:59 2012
New Revision: 1380085
URL: http://svn.apache.org/viewvc?rev=1380085&view=rev
Log:
PIG-2781: LOSort isEqual method
Modified:
pig/trunk/CHANGES.txt
pig/trunk/src/org/apache/pig/newplan/logical/relational/LOSort.java
Modified: pig/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/trunk/CHANGES.txt?rev=1380085&r1=1380084&r2=1380085&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Sun Sep 2 22:52:59 2012
@@ -24,6 +24,8 @@ INCOMPATIBLE CHANGES
IMPROVEMENTS
+PIG-2781: LOSort isEqual method (xalan via dvryaboy)
+
PIG-2835: Optimizing the convertion from bytes to Integer/Long (jay23jack via dvryaboy)
PIG-2886: Add Scan TimeRange to HBaseStorage (ted.m via dvryaboy)
Modified: pig/trunk/src/org/apache/pig/newplan/logical/relational/LOSort.java
URL: http://svn.apache.org/viewvc/pig/trunk/src/org/apache/pig/newplan/logical/relational/LOSort.java?rev=1380085&r1=1380084&r2=1380085&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/newplan/logical/relational/LOSort.java (original)
+++ pig/trunk/src/org/apache/pig/newplan/logical/relational/LOSort.java Sun Sep 2 22:52:59 2012
@@ -182,13 +182,13 @@ public class LOSort extends LogicalRelat
LOSort otherSort = (LOSort)other;
if (!mAscCols.equals(otherSort.getAscendingCols()))
return false;
- if (mSortFunc.equals(otherSort.getUserFunc()))
+ if (!mSortFunc.equals(otherSort.getUserFunc()))
return false;
if (mIsStar!=otherSort.isStar())
return false;
if (limit!=otherSort.getLimit())
return false;
- if (mSortColPlans.equals(otherSort.getSortColPlans()))
+ if (!mSortColPlans.equals(otherSort.getSortColPlans()))
return false;
}
return checkEquality((LogicalRelationalOperator)other);