Skip to content

Commit

Permalink
[testcase] Fix testcase for the picked PR apache#1565 (apache#1588)
Browse files Browse the repository at this point in the history
  • Loading branch information
@xiaokang
xiaokang authored Apr 4, 2023
1 parent 2c0077e commit 0097043
Show file tree
Hide file tree
Showing 3 changed files with 73 additions and 149 deletions.
74 changes: 0 additions & 74 deletions ...re/src/test/java/org/apache/doris/nereids/rules/analysis/CheckExpressionLegalityTest.java
View file

This file was deleted.

146 changes: 72 additions & 74 deletions regression-test/suites/materialized_view_p0/test_create_mv_complex_type.groovy
View file
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,8 @@ suite ("create_mv_complex_type") {
assertTrue(useTime <= OpTimeout, "wait_for_create_mv_finish timeout")
}

sql "ADMIN SET FRONTEND CONFIG ('enable_struct_type' = 'true')"
sql "ADMIN SET FRONTEND CONFIG ('enable_map_type' = 'true')"
// sql "ADMIN SET FRONTEND CONFIG ('enable_struct_type' = 'true')"
// sql "ADMIN SET FRONTEND CONFIG ('enable_map_type' = 'true')"

sql """ DROP TABLE IF EXISTS base_table; """
sql """
Expand All @@ -41,16 +41,14 @@ suite ("create_mv_complex_type") {
c_bigint BIGINT(10),
c_float BIGINT,
c_jsonb JSONB,
c_array ARRAY<INT>,
c_map MAP<STRING, INT>,
c_struct STRUCT<a:INT, b:INT>
c_array ARRAY<INT>
)
duplicate key (c_int)
distributed BY hash(c_int) buckets 3
properties("replication_num" = "1");
"""

sql """insert into base_table select 1, 100000, 1.0, '{"jsonk1": 123}', [100, 200], {"k1": 10}, {1, 2};"""
sql """insert into base_table select 1, 100000, 1.0, '{"jsonk1": 123}', [100, 200];"""

def success = false

Expand Down Expand Up @@ -91,41 +89,41 @@ suite ("create_mv_complex_type") {
}
assertFalse(success)

success = false
try {
sql """create materialized view mv as select c_map, c_int from base_table;"""
success = true
} catch (Exception e) {
assertTrue(e.getMessage().contains("not support to create materialized view"), e.getMessage())
}
assertFalse(success)

success = false
try {
sql """create materialized view mv as select c_bigint, c_map from base_table;"""
success = true
} catch (Exception e) {
assertTrue(e.getMessage().contains("not support to create materialized view"), e.getMessage())
}
assertFalse(success)

success = false
try {
sql """create materialized view mv as select c_struct, c_int from base_table;"""
success = true
} catch (Exception e) {
assertTrue(e.getMessage().contains("not support to create materialized view"), e.getMessage())
}
assertFalse(success)

success = false
try {
sql """create materialized view mv as select c_bigint, c_struct from base_table;"""
success = true
} catch (Exception e) {
assertTrue(e.getMessage().contains("not support to create materialized view"), e.getMessage())
}
assertFalse(success)
// success = false
// try {
// sql """create materialized view mv as select c_map, c_int from base_table;"""
// success = true
// } catch (Exception e) {
// assertTrue(e.getMessage().contains("not support to create materialized view"), e.getMessage())
// }
// assertFalse(success)

// success = false
// try {
// sql """create materialized view mv as select c_bigint, c_map from base_table;"""
// success = true
// } catch (Exception e) {
// assertTrue(e.getMessage().contains("not support to create materialized view"), e.getMessage())
// }
// assertFalse(success)

// success = false
// try {
// sql """create materialized view mv as select c_struct, c_int from base_table;"""
// success = true
// } catch (Exception e) {
// assertTrue(e.getMessage().contains("not support to create materialized view"), e.getMessage())
// }
// assertFalse(success)

// success = false
// try {
// sql """create materialized view mv as select c_bigint, c_struct from base_table;"""
// success = true
// } catch (Exception e) {
// assertTrue(e.getMessage().contains("not support to create materialized view"), e.getMessage())
// }
// assertFalse(success)


// 2. special column - mv agg key
Expand All @@ -147,23 +145,23 @@ suite ("create_mv_complex_type") {
}
assertFalse(success)

success = false
try {
sql """create materialized view mv as select c_bigint, c_int, c_map, count(c_bigint) from base_table group by c_bigint, c_int, c_map;"""
success = true
} catch (Exception e) {
assertTrue(e.getMessage().contains("don't support filter or group by"), e.getMessage())
}
assertFalse(success)
// success = false
// try {
// sql """create materialized view mv as select c_bigint, c_int, c_map, count(c_bigint) from base_table group by c_bigint, c_int, c_map;"""
// success = true
// } catch (Exception e) {
// assertTrue(e.getMessage().contains("don't support filter or group by"), e.getMessage())
// }
// assertFalse(success)

success = false
try {
sql """create materialized view mv as select c_bigint, c_int, c_struct, count(c_bigint) from base_table group by c_bigint, c_int, c_struct;"""
success = true
} catch (Exception e) {
assertTrue(e.getMessage().contains("don't support filter or group by"), e.getMessage())
}
assertFalse(success)
// success = false
// try {
// sql """create materialized view mv as select c_bigint, c_int, c_struct, count(c_bigint) from base_table group by c_bigint, c_int, c_struct;"""
// success = true
// } catch (Exception e) {
// assertTrue(e.getMessage().contains("don't support filter or group by"), e.getMessage())
// }
// assertFalse(success)


// 3. special column - ORDER BY
Expand All @@ -185,21 +183,21 @@ suite ("create_mv_complex_type") {
}
assertFalse(success)

success = false
try {
sql """create materialized view mv as select c_bigint, c_int, c_map from base_table order by c_bigint, c_int, c_map;"""
success = true
} catch (Exception e) {
assertTrue(e.getMessage().contains("don't support filter or group by"), e.getMessage())
}
assertFalse(success)

success = false
try {
sql """create materialized view mv as select c_bigint, c_int, c_struct from base_table order by c_bigint, c_int, c_struct;"""
success = true
} catch (Exception e) {
assertTrue(e.getMessage().contains("don't support filter or group by"), e.getMessage())
}
assertFalse(success)
// success = false
// try {
// sql """create materialized view mv as select c_bigint, c_int, c_map from base_table order by c_bigint, c_int, c_map;"""
// success = true
// } catch (Exception e) {
// assertTrue(e.getMessage().contains("don't support filter or group by"), e.getMessage())
// }
// assertFalse(success)

// success = false
// try {
// sql """create materialized view mv as select c_bigint, c_int, c_struct from base_table order by c_bigint, c_int, c_struct;"""
// success = true
// } catch (Exception e) {
// assertTrue(e.getMessage().contains("don't support filter or group by"), e.getMessage())
// }
// assertFalse(success)
}
2 changes: 1 addition & 1 deletion regression-test/suites/rollup_p0/test_materialized_view_array.groovy
View file
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ suite("test_materialized_view_array", "rollup") {
create_test_table.call(tableName)
test {
sql "CREATE MATERIALIZED VIEW idx AS select k1, k2, k3, k4, k5 from ${tableName}"
exception "errCode = 2, detailMessage = The array column[`k2` array<smallint(6)> NULL] not support to create materialized view"
exception "errCode = 2, detailMessage = The ARRAY column[`k2` array<smallint(6)> NULL] not support to create materialized view"
}
} finally {
try_sql("DROP TABLE IF EXISTS ${tableName}")
Expand Down

0 comments on commit 0097043

Please sign in to comment.