You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "mrhhsg (via GitHub)" <gi...@apache.org> on 2023/11/21 07:50:50 UTC
[PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
mrhhsg opened a new pull request, #27352:
URL: https://github.com/apache/doris/pull/27352
## Proposed changes
Issue Number: close #xxx
<!--Describe your changes.-->
## Further comments
If this is a relatively large or complex change, kick off the discussion at [dev@doris.apache.org](mailto:dev@doris.apache.org) by explaining why you chose the solution you did and what alternatives you considered, etc...
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "yiguolei (via GitHub)" <gi...@apache.org>.
yiguolei commented on code in PR #27352:
URL: https://github.com/apache/doris/pull/27352#discussion_r1400385521
##########
regression-test/suites/datatype_p0/decimalv2/test_decimalv2_common.groovy:
##########
@@ -0,0 +1,411 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership. The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License. You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied. See the License for the
+// specific language governing permissions and limitations
+// under the License.
+
+suite("test_decimalv2_common", "nonConcurrent") {
+
+ sql """
+ admin set frontend config("enable_decimal_conversion" = "false");
+ """
+
+ def table_normal = "test_decimalv2_common_normal_tbl"
+ def table_dup = "test_decimalv2_common_dup_tbl" // duplicate key
+ def table_uniq = "test_decimalv2_common_uniq_tbl" // unique key
+ def table_agg = "test_decimalv2_common_agg_tbl" // aggregate key
+ def table_dist = "test_decimalv2_common_dist_tbl" // distributed by
+ def table_part = "test_decimalv2_common_part_tbl"; // partition by
+
+ sql "drop table if exists ${table_dup}"
+ sql "drop table if exists ${table_uniq}"
+ sql "drop table if exists ${table_agg}"
+ sql "drop table if exists ${table_dist}"
+ sql "drop table if exists ${table_part}"
+
+ sql """
+ CREATE TABLE IF NOT EXISTS `${table_dup}` (
+ `decimal_key1` decimalv2(8, 5) NULL COMMENT "",
+ `decimal_key2` decimalv2(16, 5) NULL COMMENT "",
+ `decimal_value1` decimalv2(8, 5) NULL COMMENT "",
+ `decimal_value2` decimalv2(16, 5) NULL COMMENT "",
+ INDEX `idx_key1` (`decimal_value1`) USING BITMAP,
+ INDEX `idx_key2` (`decimal_value2`) USING BITMAP
+ ) ENGINE=OLAP
+ DUPLICATE KEY(`decimal_key1`, `decimal_key2`)
+ COMMENT "OLAP"
+ DISTRIBUTED BY HASH(`decimal_key1`, `decimal_key2`) BUCKETS 4
+ PROPERTIES (
+ "replication_allocation" = "tag.location.default: 1",
+ "in_memory" = "false",
+ "storage_format" = "V2"
+ )
+ """
+
+ sql """
+ CREATE TABLE IF NOT EXISTS `${table_uniq}` (
+ `decimal_key1` decimalv2(8, 5) NULL COMMENT "",
+ `decimal_key2` decimalv2(16, 5) NULL COMMENT "",
+ `decimal_value1` decimalv2(8, 5) NULL COMMENT "",
+ `decimal_value2` decimalv2(16, 5) NULL COMMENT "",
+ INDEX `idx_key1` (`decimal_value1`) USING BITMAP,
+ INDEX `idx_key2` (`decimal_value2`) USING BITMAP
+ ) ENGINE=OLAP
+ UNIQUE KEY(`decimal_key1`, `decimal_key2`)
+ COMMENT "OLAP"
+ DISTRIBUTED BY HASH(`decimal_key1`, `decimal_key2`) BUCKETS 4
+ PROPERTIES (
+ "replication_allocation" = "tag.location.default: 1",
+ "in_memory" = "false",
+ "storage_format" = "V2"
+ )
+ """
+
+ sql """
+ CREATE TABLE IF NOT EXISTS `${table_agg}` (
+ `decimal_key1` decimalv2(8, 5) NULL COMMENT "",
+ `decimal_key2` decimalv2(16, 5) NULL COMMENT "",
+ `decimal_value1` decimalv2(8, 5) sum NULL COMMENT "",
+ `decimal_value2` decimalv2(16, 5) max NULL COMMENT ""
+ ) ENGINE=OLAP
+ AGGREGATE KEY(`decimal_key1`, `decimal_key2`)
+ COMMENT "OLAP"
+ DISTRIBUTED BY HASH(`decimal_key1`, `decimal_key2`) BUCKETS 4
+ PROPERTIES (
+ "replication_allocation" = "tag.location.default: 1",
+ "in_memory" = "false",
+ "storage_format" = "V2"
+ )
+ """
+
+ sql """
+ CREATE TABLE IF NOT EXISTS `${table_dist}` (
+ `decimal_key1` decimalv2(8, 5) NULL COMMENT "",
+ `decimal_key2` decimalv2(16, 5) NULL COMMENT "",
+ `decimal_value1` decimalv2(8, 5) NULL COMMENT "",
+ `decimal_value2` decimalv2(16, 5) NULL COMMENT "",
+ INDEX `idx_key1` (`decimal_value1`) USING BITMAP,
+ INDEX `idx_key2` (`decimal_value2`) USING BITMAP
+ ) ENGINE=OLAP
+ DUPLICATE KEY(`decimal_key1`, `decimal_key2`)
+ COMMENT "OLAP"
+ DISTRIBUTED BY HASH(`decimal_key1`, `decimal_key2`) BUCKETS 4
+ PROPERTIES (
+ "replication_allocation" = "tag.location.default: 1",
+ "in_memory" = "false",
+ "storage_format" = "V2"
+ )
+ """
+
+ /*** TODO: Column[decimal_key1] type[decimalv2] cannot be a range partition key.
+ sql """
+ CREATE TABLE IF NOT EXISTS `${table_part}` (
+ `decimal_key1` decimalv2(8, 5) NULL COMMENT "",
+ `decimal_key2` decimalv2(16, 5) NULL COMMENT "",
+ `decimal_value1` decimalv2(8, 5) NULL COMMENT "",
+ `decimal_value2` decimalv2(16, 5) NULL COMMENT "",
+ INDEX `idx_key1` (`decimal_value1`) USING BITMAP,
+ INDEX `idx_key2` (`decimal_value2`) USING BITMAP
+ ) ENGINE=OLAP
+ UNIQUE KEY(`decimal_key1`, `decimal_key2`)
+ PARTITION BY RANGE(`decimal_key1`) (
+ PARTITION `p1` VALUES LESS THAN ('1.1111'),
+ PARTITION `p2` VALUES LESS THAN ('2.2222'),
+ PARTITION `p3` VALUES LESS THAN ('3.3333'),
+ PARTITION `p4` VALUES LESS THAN ('4.4444'),
+ PARTITION `p5` VALUES LESS THAN ('5.5555'),
+ PARTITION `p6` VALUES LESS THAN ('100.9999')
+ )
+ DISTRIBUTED BY HASH(`decimal_key1`) BUCKETS 2
+ PROPERTIES (
+ "replication_allocation" = "tag.location.default: 1",
+ "in_memory" = "false",
+ "storage_format" = "V2"
+ )
+ """
+ */
+
+ def insert_data = { table_name ->
+ sql """
+ insert into ${table_name}
+ values
+ (1.1111, 1.1111, 1.1111, 1.1111),
+ (2.2222, 1.1111, 1.1111, 1.1111),
+ (3.3333, 1.1111, 1.1111, 1.1111),
+ (4.4444, 1.1111, 1.1111, 1.1111),
+ (5.5555, 1.1111, 1.1111, 1.1111),
+ (6.6666, 1.1111, 1.1111, 1.1111),
+ (7.7777, 1.1111, 1.1111, 1.1111),
+ (8.8888, 1.1111, 1.1111, 1.1111),
+ (9.9999, 1.1111, 1.1111, 1.1111);
+ """
+ sql """
+ insert into ${table_name}
+ values
+ (1.1111, 1.1111, 1.1111, 1.1111),
+ (2.2222, 2.2222, 1.1111, 1.1111),
+ (3.3333, 3.3333, 1.1111, 1.1111),
+ (4.4444, 4.4444, 1.1111, 1.1111),
+ (5.5555, 5.5555, 1.1111, 1.1111),
+ (6.6666, 6.6666, 1.1111, 1.1111),
+ (7.7777, 7.7777, 1.1111, 1.1111),
+ (8.8888, 8.8888, 1.1111, 1.1111),
+ (9.9999, 9.9999, 1.1111, 1.1111);
+ """
+ sql """
+ insert into ${table_name}
+ values
+ (1.1111, 1.1111, 1.1111, 1.1111),
+ (2.2222, 2.2222, 2.2222, 1.1111),
+ (3.3333, 3.3333, 3.3333, 1.1111),
+ (4.4444, 4.4444, 4.4444, 1.1111),
+ (5.5555, 5.5555, 5.5555, 1.1111),
+ (6.6666, 6.6666, 6.6666, 1.1111),
+ (7.7777, 7.7777, 7.7777, 1.1111),
+ (8.8888, 8.8888, 8.8888, 1.1111),
+ (9.9999, 9.9999, 9.9999, 1.1111);
+ """
+ sql """
+ insert into ${table_name}
+ values
+ (1.1111, 1.1111, 1.1111, 1.1111),
+ (2.2222, 2.2222, 2.2222, 2.2222),
+ (3.3333, 3.3333, 3.3333, 3.3333),
+ (4.4444, 4.4444, 4.4444, 4.4444),
+ (5.5555, 5.5555, 5.5555, 5.5555),
+ (6.6666, 6.6666, 6.6666, 6.6666),
+ (7.7777, 7.7777, 7.7777, 7.7777),
+ (8.8888, 8.8888, 8.8888, 8.8888),
+ (9.9999, 9.9999, 9.9999, 9.9999);
+ """
+ sql """
+ insert into ${table_name}
+ values
+ (null, 1.1111, 1.1111, 1.1111),
+ (2.2222, null, 2.2222, 2.2222),
+ (3.3333, 3.3333, null, 3.3333),
+ (4.4444, 4.4444, 4.4444, null),
+ (5.5555, 5.5555, 5.5555, null),
+ (6.6666, 6.6666, null, 6.6666),
+ (7.7777, null, 7.7777, 7.7777),
+ (null, 8.8888, 8.8888, 8.8888),
+ (9.9999, null, 9.9999, 9.9999);
+ """
+ sql """
+ insert into ${table_name}
+ values
+ (1.1111, null, 1.1111, 1.1111),
+ (2.2222, 2.2222, null, 2.2222),
+ (3.3333, 3.3333, 3.3333, null),
+ (null, 4.4444, 4.4444, 4.4444),
+ (5.5555, null, 5.5555, 5.5555),
+ (6.6666, 6.6666, null, 6.6666),
+ (7.7777, 7.7777, 7.7777, null),
+ (null, 8.8888, 8.8888, 8.8888),
+ (9.9999, null, 9.9999, 9.9999);
+ """
+ }
+
+ insert_data(table_dup)
+ insert_data(table_agg)
+ insert_data(table_uniq)
+ insert_data(table_dist)
+ // insert_data(table_part)
+
+ def run_predicate_test = { table_name, col, col_value, bitmap_col, bitmap_col_value, in_list ->
+ def query = """
Review Comment:
add comment here to explain test which case
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27352:
URL: https://github.com/apache/doris/pull/27352#issuecomment-1820443214
TeamCity be ut coverage result:
Function Coverage: 36.53% (8447/23125)
Line Coverage: 28.86% (68672/237936)
Region Coverage: 27.83% (35518/127637)
Branch Coverage: 24.57% (18107/73682)
Coverage Report: http://coverage.selectdb-in.cc/coverage/07d320b5ecbde284e122e1848a591a048412ae65_07d320b5ecbde284e122e1848a591a048412ae65/report/index.html
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #27352:
URL: https://github.com/apache/doris/pull/27352#issuecomment-1820613360
clang-tidy review says "All clean, LGTM! :+1:"
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #27352:
URL: https://github.com/apache/doris/pull/27352#issuecomment-1820575191
clang-tidy review says "All clean, LGTM! :+1:"
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "mrhhsg (via GitHub)" <gi...@apache.org>.
mrhhsg commented on PR #27352:
URL: https://github.com/apache/doris/pull/27352#issuecomment-1820600405
run buildall
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #27352:
URL: https://github.com/apache/doris/pull/27352#issuecomment-1820405870
clang-tidy review says "All clean, LGTM! :+1:"
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "mrhhsg (via GitHub)" <gi...@apache.org>.
mrhhsg commented on PR #27352:
URL: https://github.com/apache/doris/pull/27352#issuecomment-1820397920
run buildall
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27352:
URL: https://github.com/apache/doris/pull/27352#issuecomment-1820747484
TeamCity be ut coverage result:
Function Coverage: 36.52% (8445/23125)
Line Coverage: 28.85% (68653/237938)
Region Coverage: 27.81% (35495/127639)
Branch Coverage: 24.57% (18103/73684)
Coverage Report: http://coverage.selectdb-in.cc/coverage/d2d8a155aa971c4c701154db7c1a82082c61119e_d2d8a155aa971c4c701154db7c1a82082c61119e/report/index.html
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #27352:
URL: https://github.com/apache/doris/pull/27352#issuecomment-1820939422
PR approved by anyone and no changes requested.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27352:
URL: https://github.com/apache/doris/pull/27352#issuecomment-1820781252
<details>
<summary>TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'</summary>
```
Tpch sf100 test result on commit d2d8a155aa971c4c701154db7c1a82082c61119e, data reload: false
run tpch-sf100 query with default conf and session variables
q1 4969 4669 4637 4637
q2 356 171 164 164
q3 2031 1976 1859 1859
q4 1382 1262 1275 1262
q5 3930 3954 4015 3954
q6 252 133 133 133
q7 1400 894 890 890
q8 2735 2766 2739 2739
q9 9787 9705 9510 9510
q10 3472 3527 3550 3527
q11 376 247 250 247
q12 445 294 299 294
q13 4585 3859 3802 3802
q14 323 288 284 284
q15 577 522 524 522
q16 674 590 582 582
q17 1124 958 947 947
q18 7764 7329 7339 7329
q19 1666 1678 1679 1678
q20 527 306 307 306
q21 4410 3934 3964 3934
q22 476 367 369 367
Total cold run time: 53261 ms
Total hot run time: 48967 ms
run tpch-sf100 query with default conf and set session variable runtime_filter_mode=off
q1 4606 4576 4548 4548
q2 337 235 252 235
q3 4030 3986 3998 3986
q4 2698 2706 2704 2704
q5 9718 9697 9727 9697
q6 237 124 122 122
q7 3008 2456 2477 2456
q8 4439 4516 4441 4441
q9 13118 12981 12972 12972
q10 4073 4162 4199 4162
q11 791 722 658 658
q12 978 815 819 815
q13 4289 3547 3615 3547
q14 374 342 351 342
q15 583 536 532 532
q16 750 674 662 662
q17 3850 3939 3891 3891
q18 9536 9013 8852 8852
q19 1792 1793 1768 1768
q20 2377 2099 2029 2029
q21 8827 8639 8709 8639
q22 855 802 814 802
Total cold run time: 81266 ms
Total hot run time: 77860 ms
```
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "mrhhsg (via GitHub)" <gi...@apache.org>.
mrhhsg commented on PR #27352:
URL: https://github.com/apache/doris/pull/27352#issuecomment-1820565728
run buildall
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "yiguolei (via GitHub)" <gi...@apache.org>.
yiguolei commented on code in PR #27352:
URL: https://github.com/apache/doris/pull/27352#discussion_r1400389616
##########
regression-test/suites/datatype_p0/decimalv2/test_decimalv2_common.groovy:
##########
@@ -0,0 +1,411 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership. The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License. You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied. See the License for the
+// specific language governing permissions and limitations
+// under the License.
+
+suite("test_decimalv2_common", "nonConcurrent") {
+
+ sql """
+ admin set frontend config("enable_decimal_conversion" = "false");
+ """
+
+ def table_normal = "test_decimalv2_common_normal_tbl"
+ def table_dup = "test_decimalv2_common_dup_tbl" // duplicate key
+ def table_uniq = "test_decimalv2_common_uniq_tbl" // unique key
+ def table_agg = "test_decimalv2_common_agg_tbl" // aggregate key
+ def table_dist = "test_decimalv2_common_dist_tbl" // distributed by
+ def table_part = "test_decimalv2_common_part_tbl"; // partition by
+
+ sql "drop table if exists ${table_dup}"
+ sql "drop table if exists ${table_uniq}"
+ sql "drop table if exists ${table_agg}"
+ sql "drop table if exists ${table_dist}"
+ sql "drop table if exists ${table_part}"
+
+ sql """
+ CREATE TABLE IF NOT EXISTS `${table_dup}` (
+ `decimal_key1` decimalv2(8, 5) NULL COMMENT "",
+ `decimal_key2` decimalv2(16, 5) NULL COMMENT "",
+ `decimal_value1` decimalv2(8, 5) NULL COMMENT "",
+ `decimal_value2` decimalv2(16, 5) NULL COMMENT "",
+ INDEX `idx_key1` (`decimal_value1`) USING BITMAP,
+ INDEX `idx_key2` (`decimal_value2`) USING BITMAP
+ ) ENGINE=OLAP
+ DUPLICATE KEY(`decimal_key1`, `decimal_key2`)
+ COMMENT "OLAP"
+ DISTRIBUTED BY HASH(`decimal_key1`, `decimal_key2`) BUCKETS 4
+ PROPERTIES (
+ "replication_allocation" = "tag.location.default: 1",
+ "in_memory" = "false",
+ "storage_format" = "V2"
+ )
+ """
+
+ sql """
+ CREATE TABLE IF NOT EXISTS `${table_uniq}` (
+ `decimal_key1` decimalv2(8, 5) NULL COMMENT "",
+ `decimal_key2` decimalv2(16, 5) NULL COMMENT "",
+ `decimal_value1` decimalv2(8, 5) NULL COMMENT "",
+ `decimal_value2` decimalv2(16, 5) NULL COMMENT "",
+ INDEX `idx_key1` (`decimal_value1`) USING BITMAP,
+ INDEX `idx_key2` (`decimal_value2`) USING BITMAP
+ ) ENGINE=OLAP
+ UNIQUE KEY(`decimal_key1`, `decimal_key2`)
+ COMMENT "OLAP"
+ DISTRIBUTED BY HASH(`decimal_key1`, `decimal_key2`) BUCKETS 4
+ PROPERTIES (
+ "replication_allocation" = "tag.location.default: 1",
+ "in_memory" = "false",
+ "storage_format" = "V2"
+ )
+ """
+
+ sql """
+ CREATE TABLE IF NOT EXISTS `${table_agg}` (
+ `decimal_key1` decimalv2(8, 5) NULL COMMENT "",
+ `decimal_key2` decimalv2(16, 5) NULL COMMENT "",
+ `decimal_value1` decimalv2(8, 5) sum NULL COMMENT "",
+ `decimal_value2` decimalv2(16, 5) max NULL COMMENT ""
+ ) ENGINE=OLAP
+ AGGREGATE KEY(`decimal_key1`, `decimal_key2`)
+ COMMENT "OLAP"
+ DISTRIBUTED BY HASH(`decimal_key1`, `decimal_key2`) BUCKETS 4
+ PROPERTIES (
+ "replication_allocation" = "tag.location.default: 1",
+ "in_memory" = "false",
+ "storage_format" = "V2"
+ )
+ """
+
+ sql """
+ CREATE TABLE IF NOT EXISTS `${table_dist}` (
+ `decimal_key1` decimalv2(8, 5) NULL COMMENT "",
+ `decimal_key2` decimalv2(16, 5) NULL COMMENT "",
+ `decimal_value1` decimalv2(8, 5) NULL COMMENT "",
+ `decimal_value2` decimalv2(16, 5) NULL COMMENT "",
+ INDEX `idx_key1` (`decimal_value1`) USING BITMAP,
+ INDEX `idx_key2` (`decimal_value2`) USING BITMAP
+ ) ENGINE=OLAP
+ DUPLICATE KEY(`decimal_key1`, `decimal_key2`)
+ COMMENT "OLAP"
+ DISTRIBUTED BY HASH(`decimal_key1`, `decimal_key2`) BUCKETS 4
+ PROPERTIES (
+ "replication_allocation" = "tag.location.default: 1",
Review Comment:
disable auto compaction please
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27352:
URL: https://github.com/apache/doris/pull/27352#issuecomment-1820799714
(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.38 seconds
stream load tsv: 579 seconds loaded 74807831229 Bytes, about 123 MB/s
stream load json: 18 seconds loaded 2358488459 Bytes, about 124 MB/s
stream load orc: 65 seconds loaded 1101869774 Bytes, about 16 MB/s
stream load parquet: 34 seconds loaded 861443392 Bytes, about 24 MB/s
insert into select: 27.9 seconds inserted 10000000 Rows, about 358K ops/s
storage size: 17099579332 Bytes
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #27352:
URL: https://github.com/apache/doris/pull/27352#issuecomment-1820939328
PR approved by at least one committer and no changes requested.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27352:
URL: https://github.com/apache/doris/pull/27352#issuecomment-1820646410
(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 44.32 seconds
stream load tsv: 574 seconds loaded 74807831229 Bytes, about 124 MB/s
stream load json: 18 seconds loaded 2358488459 Bytes, about 124 MB/s
stream load orc: 65 seconds loaded 1101869774 Bytes, about 16 MB/s
stream load parquet: 34 seconds loaded 861443392 Bytes, about 24 MB/s
insert into select: 28.7 seconds inserted 10000000 Rows, about 348K ops/s
storage size: 17098753647 Bytes
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27352:
URL: https://github.com/apache/doris/pull/27352#issuecomment-1820582813
(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 44.7 seconds
stream load tsv: 572 seconds loaded 74807831229 Bytes, about 124 MB/s
stream load json: 18 seconds loaded 2358488459 Bytes, about 124 MB/s
stream load orc: 65 seconds loaded 1101869774 Bytes, about 16 MB/s
stream load parquet: 34 seconds loaded 861443392 Bytes, about 24 MB/s
insert into select: 28.7 seconds inserted 10000000 Rows, about 348K ops/s
storage size: 17097001829 Bytes
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #27352:
URL: https://github.com/apache/doris/pull/27352#issuecomment-1820591945
TeamCity be ut coverage result:
Function Coverage: 36.53% (8448/23125)
Line Coverage: 28.87% (68687/237936)
Region Coverage: 27.84% (35528/127637)
Branch Coverage: 24.59% (18117/73682)
Coverage Report: http://coverage.selectdb-in.cc/coverage/42c9d1e21e192973ecb9e0117cccccd82909b0cf_42c9d1e21e192973ecb9e0117cccccd82909b0cf/report/index.html
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [test](decimalv2) add some regression cases about decimalv2 [doris]
Posted by "yiguolei (via GitHub)" <gi...@apache.org>.
yiguolei merged PR #27352:
URL: https://github.com/apache/doris/pull/27352
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org