You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by yc...@apache.org on 2016/02/17 21:51:48 UTC
hive git commit: HIVE-13039: BETWEEN predicate is not functioning
correctly with predicate pushdown on Parquet table. More fixes for tests
(Yongzhi Chen, reviewed by Sergio Pena)
Repository: hive
Updated Branches:
refs/heads/branch-1 3dd13b1bc -> 23a963698
HIVE-13039: BETWEEN predicate is not functioning correctly with predicate pushdown on Parquet table. More fixes for tests (Yongzhi Chen, reviewed by Sergio Pena)
Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/23a96369
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/23a96369
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/23a96369
Branch: refs/heads/branch-1
Commit: 23a963698355ce4b62ab2af3b7c6ce04b1244d13
Parents: 3dd13b1
Author: Yongzhi Chen <yc...@apache.org>
Authored: Wed Feb 17 11:23:41 2016 -0500
Committer: Yongzhi Chen <yc...@apache.org>
Committed: Wed Feb 17 15:51:02 2016 -0500
----------------------------------------------------------------------
.../hadoop/hive/ql/io/sarg/TestSearchArgumentImpl.java | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hive/blob/23a96369/ql/src/test/org/apache/hadoop/hive/ql/io/sarg/TestSearchArgumentImpl.java
----------------------------------------------------------------------
diff --git a/ql/src/test/org/apache/hadoop/hive/ql/io/sarg/TestSearchArgumentImpl.java b/ql/src/test/org/apache/hadoop/hive/ql/io/sarg/TestSearchArgumentImpl.java
index 9e6adef..576e677 100644
--- a/ql/src/test/org/apache/hadoop/hive/ql/io/sarg/TestSearchArgumentImpl.java
+++ b/ql/src/test/org/apache/hadoop/hive/ql/io/sarg/TestSearchArgumentImpl.java
@@ -1500,8 +1500,8 @@ public class TestSearchArgumentImpl {
assertEquals(3, leaves.size());
String[] conditions = new String[]{
- "lt(id, 45)", /* id between 23 and 45 */
- "not(lteq(id, 23))", /* id between 23 and 45 */
+ "lteq(id, 45)", /* id between 23 and 45 */
+ "not(lt(id, 23))", /* id between 23 and 45 */
"eq(first_name, Binary{\"alan\"})", /* first_name = 'alan' */
"eq(last_name, Binary{\"smith\"})" /* 'smith' = last_name */
};
@@ -1988,7 +1988,7 @@ public class TestSearchArgumentImpl {
FilterPredicate p = sarg.toFilterPredicate();
String expected =
- "and(lt(first_name, Binary{\"greg\"}), not(lteq(first_name, Binary{\"david\"})))";
+ "and(lteq(first_name, Binary{\"greg\"}), not(lt(first_name, Binary{\"david\"})))";
assertEquals(p.toString(), expected);
assertEquals(PredicateLeaf.Type.STRING, leaves.get(0).getType());
@@ -2936,7 +2936,7 @@ public class TestSearchArgumentImpl {
FilterPredicate p = sarg.toFilterPredicate();
String expected =
- "and(and(and(not(eq(x, null)), not(and(lt(y, 20), not(lteq(y, 10))))), not(or(or(eq(z, 1), " +
+ "and(and(and(not(eq(x, null)), not(and(lteq(y, 20), not(lt(y, 10))))), not(or(or(eq(z, 1), " +
"eq(z, 2)), eq(z, 3)))), not(eq(a, Binary{\"stinger\"})))";
assertEquals(expected, p.toString());
}