Skip to content

Commit 1210995

Browse files
shtilmanandrewseidl
authored andcommitted
Adjust expected counts due to switch to compressed cIntersects
1 parent 85e049c commit 1210995

File tree

1 file changed

+10
-9
lines changed

1 file changed

+10
-9
lines changed

Tests/OverlapsJoinTest.cpp

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -296,7 +296,8 @@ TEST_F(OverlapsTest, InnerJoinPolyInPointIntersects) {
296296
} else {
297297
// Note(jclay): We return 0, postgis returns 4
298298
// Note(adb): Now we return 3. Progress?
299-
ASSERT_EQ(static_cast<int64_t>(3), v<int64_t>(execSQL(sql, ctx.device_type)));
299+
// Note(ds): After switching to cIntersects we return 0 again. Progress?
300+
ASSERT_EQ(static_cast<int64_t>(0), v<int64_t>(execSQL(sql, ctx.device_type)));
300301
}
301302
});
302303
}
@@ -1576,8 +1577,8 @@ TEST_F(MultiFragGeoOverlapsJoinTest, Nullable_Geo_Exhaustive) {
15761577
executeAllScenarios([](const ExecutionContext ctx) -> void {
15771578
int64_t single_frag_res1 = 114;
15781579
int64_t single_frag_res2 = 5163;
1579-
int64_t single_frag_res3 = 2178;
1580-
int64_t single_frag_res4 = 2178;
1580+
int64_t single_frag_res3 = 2144;
1581+
int64_t single_frag_res4 = 2144;
15811582
std::ostringstream mq1, mq2, mq3, mq4;
15821583
mq1 << "SELECT COUNT(1) FROM mfgeo_n_v2 r, mfgeo_n_v2 s WHERE ST_INTERSECTS(r.pt, "
15831584
"s.pt);";
@@ -1612,8 +1613,8 @@ TEST_F(ParallelLinearization, GeoJoin) {
16121613
executeAllScenarios([](const ExecutionContext ctx) -> void {
16131614
int64_t single_frag_res1 = 1020;
16141615
int64_t single_frag_res2 = 80940;
1615-
int64_t single_frag_res3 = 38378;
1616-
int64_t single_frag_res4 = 38378;
1616+
int64_t single_frag_res3 = 38080;
1617+
int64_t single_frag_res4 = 38080;
16171618
std::ostringstream mq1, mq2, mq3, mq4;
16181619
mq1 << "SELECT COUNT(1) FROM mfgeo_p r, mfgeo_p s WHERE ST_INTERSECTS(r.pt, s.pt);";
16191620
mq2 << "SELECT COUNT(1) FROM mfgeo_p r, mfgeo_p s WHERE ST_INTERSECTS(s.p, r.l);";
@@ -1636,8 +1637,8 @@ TEST_F(ParallelLinearization, GeoJoin) {
16361637
executeAllScenarios([](const ExecutionContext ctx) -> void {
16371638
int64_t single_frag_res1 = 895;
16381639
int64_t single_frag_res2 = 70115;
1639-
int64_t single_frag_res3 = 33096;
1640-
int64_t single_frag_res4 = 33096;
1640+
int64_t single_frag_res3 = 32820;
1641+
int64_t single_frag_res4 = 32820;
16411642
std::ostringstream mq1, mq2, mq3, mq4;
16421643
mq1 << "SELECT COUNT(1) FROM mfgeo_n_p r, mfgeo_n_p s WHERE ST_INTERSECTS(r.pt, "
16431644
"s.pt);";
@@ -1663,8 +1664,8 @@ TEST_F(ParallelLinearization, GeoJoin) {
16631664
executeAllScenarios([](const ExecutionContext ctx) -> void {
16641665
int64_t single_frag_res1 = 914;
16651666
int64_t single_frag_res2 = 71556;
1666-
int64_t single_frag_res3 = 33905;
1667-
int64_t single_frag_res4 = 33905;
1667+
int64_t single_frag_res3 = 33619;
1668+
int64_t single_frag_res4 = 33619;
16681669
std::ostringstream mq1, mq2, mq3, mq4;
16691670
mq1 << "SELECT COUNT(1) FROM mfgeo_n2_p r, mfgeo_n2_p s WHERE ST_INTERSECTS(r.pt, "
16701671
"s.pt);";

0 commit comments

Comments
 (0)