NOT operator fixed

zilder · zilder · commit d586756fc5ce · 2016-11-07T12:11:49.000+03:00
diff --git a/expected/pathman_basic.out b/expected/pathman_basic.out
@@ -1428,6 +1428,41 @@ EXPLAIN (COSTS OFF) SELECT * FROM messages;
    ->  Seq Scan on messages_2
 (3 rows)
 
+/* Testing NOT operator */
+CREATE TABLE bool_test(a serial, b BOOLEAN);
+SELECT create_hash_partitions('bool_test', 'a', 3);
+ create_hash_partitions 
+------------------------
+                      3
+(1 row)
+
+INSERT INTO bool_test SELECT g, (g % 2) = 0 FROM GENERATE_SERIES(1, 100) AS g;
+SELECT count(*) FROM bool_test;
+ count 
+-------
+   100
+(1 row)
+
+SELECT count(*) FROM bool_test WHERE (b = true AND b = false);
+ count 
+-------
+     0
+(1 row)
+
+SELECT count(*) FROM bool_test WHERE b = false;
+ count 
+-------
+    50
+(1 row)
+
+SELECT count(*) FROM bool_test WHERE b = false;
+ count 
+-------
+    50
+(1 row)
+
+DROP TABLE bool_test CASCADE;
+NOTICE:  drop cascades to 3 other objects
 DROP SCHEMA test CASCADE;
 NOTICE:  drop cascades to 13 other objects
 DROP EXTENSION pg_pathman CASCADE;
diff --git a/sql/pathman_basic.sql b/sql/pathman_basic.sql
@@ -387,6 +387,15 @@ ALTER TABLE replies DROP CONSTRAINT replies_message_id_fkey;
 SELECT create_range_partitions('messages', 'id', 1, 100, 2);
 EXPLAIN (COSTS OFF) SELECT * FROM messages;
 
+/* Testing NOT operator */
+CREATE TABLE bool_test(a serial, b BOOLEAN);
+SELECT create_hash_partitions('bool_test', 'a', 3);
+INSERT INTO bool_test SELECT g, (g % 2) = 0 FROM GENERATE_SERIES(1, 100) AS g;
+SELECT count(*) FROM bool_test;
+SELECT count(*) FROM bool_test WHERE (b = true AND b = false);
+SELECT count(*) FROM bool_test WHERE b = false;
+SELECT count(*) FROM bool_test WHERE b = false;
+DROP TABLE bool_test CASCADE;
 
 DROP SCHEMA test CASCADE;
 DROP EXTENSION pg_pathman CASCADE;
diff --git a/src/pg_pathman.c b/src/pg_pathman.c
@@ -1533,7 +1533,7 @@ handle_boolexpr(const BoolExpr *expr, WalkerContext *context)
 			default:
 				result->rangeset = list_make1_irange(make_irange(0,
 																 PrelLastChild(prel),
-																 false));
+																 true));
 				break;
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -1533,7 +1533,7 @@ handle_boolexpr(const BoolExpr expr, WalkerContext context)`
`1533`	`1533`	`default:`
`1534`	`1534`	`result->rangeset = list_make1_irange(make_irange(0,`
`1535`	`1535`	`PrelLastChild(prel),`
`1536`		`- false));`
	`1536`	`+ true));`
`1537`	`1537`	`break;`
`1538`	`1538`	`}`
`1539`	`1539`	`}`