You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "amorynan (via GitHub)" <gi...@apache.org> on 2023/10/12 08:22:26 UTC
[PR] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
amorynan opened a new pull request, #25355:
URL: https://github.com/apache/doris/pull/25355
## Proposed changes
Issue Number: close #xxx
this pr
1. fix use podarray push_back() with back() will make heap_use_after_free when podarray is reach capacity which would may make heap free
2. add cases for csv format for nested types. and csv file has two define which are without quote or just like json text
<!--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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1759284700
(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.79 seconds
stream load tsv: 557 seconds loaded 74807831229 Bytes, about 128 MB/s
stream load json: 20 seconds loaded 2358488459 Bytes, about 112 MB/s
stream load orc: 64 seconds loaded 1101869774 Bytes, about 16 MB/s
stream load parquet: 32 seconds loaded 861443392 Bytes, about 25 MB/s
insert into select: 29.0 seconds inserted 10000000 Rows, about 344K ops/s
storage size: 17162008248 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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1759168316
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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1761269354
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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "amorynan (via GitHub)" <gi...@apache.org>.
amorynan commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1759166434
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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1759266291
TeamCity be ut coverage result:
Function Coverage: 36.24% (8156/22508)
Line Coverage: 28.34% (65296/230406)
Region Coverage: 27.05% (33811/124997)
Branch Coverage: 23.88% (17256/72258)
Coverage Report: http://coverage.selectdb-in.cc/coverage/586aa5959efde16ff29e529c8034224afbec6b64_586aa5959efde16ff29e529c8034224afbec6b64/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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "amorynan (via GitHub)" <gi...@apache.org>.
amorynan commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1761259901
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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1760750012
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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "amorynan (via GitHub)" <gi...@apache.org>.
amorynan commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1760744867
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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1761302661
TeamCity be ut coverage result:
Function Coverage: 36.28% (8144/22450)
Line Coverage: 28.50% (65290/229095)
Region Coverage: 27.18% (33811/124403)
Branch Coverage: 23.98% (17253/71942)
Coverage Report: http://coverage.selectdb-in.cc/coverage/da86d1287d8091e13b8200ecd86a376b8e01d878_da86d1287d8091e13b8200ecd86a376b8e01d878/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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1760992963
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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1760758387
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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1759198513
TeamCity be ut coverage result:
Function Coverage: 36.23% (8154/22507)
Line Coverage: 28.34% (65289/230390)
Region Coverage: 27.05% (33813/124988)
Branch Coverage: 23.88% (17252/72254)
Coverage Report: http://coverage.selectdb-in.cc/coverage/f0cfed9d0b0c2934ddd880273d48d361ded423cd_f0cfed9d0b0c2934ddd880273d48d361ded423cd/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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1760987406
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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1760810884
(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.7 seconds
stream load tsv: 555 seconds loaded 74807831229 Bytes, about 128 MB/s
stream load json: 20 seconds loaded 2358488459 Bytes, about 112 MB/s
stream load orc: 64 seconds loaded 1101869774 Bytes, about 16 MB/s
stream load parquet: 32 seconds loaded 861443392 Bytes, about 25 MB/s
insert into select: 28.9 seconds inserted 10000000 Rows, about 346K ops/s
storage size: 17161970000 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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "amorynan (via GitHub)" <gi...@apache.org>.
amorynan commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1760974427
run p0
--
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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1759284266
(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.8 seconds
stream load tsv: 561 seconds loaded 74807831229 Bytes, about 127 MB/s
stream load json: 20 seconds loaded 2358488459 Bytes, about 112 MB/s
stream load orc: 64 seconds loaded 1101869774 Bytes, about 16 MB/s
stream load parquet: 33 seconds loaded 861443392 Bytes, about 24 MB/s
insert into select: 28.6 seconds inserted 10000000 Rows, about 349K ops/s
storage size: 17162124764 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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1760750078
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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "morningman (via GitHub)" <gi...@apache.org>.
morningman merged PR #25355:
URL: https://github.com/apache/doris/pull/25355
--
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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1760823177
TeamCity be ut coverage result:
Function Coverage: 36.29% (8145/22446)
Line Coverage: 28.42% (65259/229591)
Region Coverage: 27.10% (33801/124726)
Branch Coverage: 23.89% (17251/72202)
Coverage Report: http://coverage.selectdb-in.cc/coverage/3ab7d866c6a22da556369480f4f9888372e666e2_3ab7d866c6a22da556369480f4f9888372e666e2/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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "amorynan (via GitHub)" <gi...@apache.org>.
amorynan commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1759159044
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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1759177982
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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "xiaokang (via GitHub)" <gi...@apache.org>.
xiaokang commented on code in PR #25355:
URL: https://github.com/apache/doris/pull/25355#discussion_r1357738498
##########
be/src/vec/data_types/serde/data_type_array_serde.cpp:
##########
@@ -94,7 +94,8 @@ Status DataTypeArraySerDe::deserialize_one_cell_from_json(IColumn& column, Slice
}
// empty array []
if (slice.size == 2) {
- offsets.push_back(offsets.back());
+ auto last_off = offsets.back();
Review Comment:
fix similar issues in another pr
##########
regression-test/suites/datatype_p0/nested_types/query/test_nestedtypes_csv_insert_into_with_s3.groovy:
##########
@@ -0,0 +1,294 @@
+// 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.
+
+
+import com.google.common.collect.Lists
+import org.apache.commons.lang3.StringUtils
+import org.codehaus.groovy.runtime.IOGroovyMethods
+
+suite("test_nestedtypes_csv_insert_into_with_s3", "p0") {
+ sql 'use regression_test_datatype_p0_nested_types'
+ sql 'set enable_nereids_planner=false'
+ sql 'set max_allowed_packet=4194304'
+ sql """ADMIN SET FRONTEND CONFIG ('disable_nested_complex_type' = 'false')"""
+
+ String ak = getS3AK()
+ String sk = getS3SK()
+ String s3_endpoint = getS3Endpoint()
+ String bucket = context.config.otherConfigs.get("s3BucketName");
+
+
+ def dataFilePath = "https://"+"${bucket}"+"."+"${s3_endpoint}"+"/regression/datalake"
+
+ // csv format define in two method:
+ // 1.without quote, like doris mysql select output,
+ // 2.csv as json format, which is different from first if meet datetime type, because json has no datetime
+ ArrayList<String> csv_without_quote = ["${dataFilePath}/as_without_quote.csv", "${dataFilePath}/arrarr_without_quote.csv",
+ "${dataFilePath}/map_without_quote.csv","${dataFilePath}/arrmap_without_quote.csv",
+ "${dataFilePath}/maparr_without_quote.csv"]
+ ArrayList<String> csv_as_json = ["${dataFilePath}/as_as_json.csv", "${dataFilePath}/arrarr_as_json.csv",
+ "${dataFilePath}/map_as_json.csv","${dataFilePath}/arrmap_as_json.csv",
+ "${dataFilePath}/maparr_as_json.csv"]
+
+
+ // define dup key table with nested table types with one nested scala
+ List<String> table_names = new ArrayList<>()
+ def nested_table_dup = "tbl_csv_array_nested_types_s3"
+ sql "DROP TABLE IF EXISTS ${nested_table_dup}"
+ sql """
+ CREATE TABLE IF NOT EXISTS ${nested_table_dup} (
+ `k1` bigint(11) NULL,
+ `c_bool` array<boolean> NULL,
+ `c_tinyint` array<tinyint(4)> NULL,
+ `c_smallint` array<smallint(6)> NULL,
+ `c_int` array<int(11)> NULL,
+ `c_bigint` array<bigint(20)> NULL,
+ `c_largeint` array<largeint(40)> NULL,
+ `c_float` array<float> NULL,
+ `c_double` array<double> NULL,
+ `c_decimal` array<decimal(20, 3)> NULL,
+ `c_decimalv3` array<decimalv3(20, 3)> NULL,
+ `c_date` array<date> NULL,
+ `c_datetime` array<datetime> NULL,
+ `c_datev2` array<datev2> NULL,
+ `c_datetimev2` array<datetimev2(0)> NULL,
+ `c_char` array<char(15)> NULL,
+ `c_varchar` array<varchar(100)> NULL,
+ `c_string` array<text> NULL
+ ) ENGINE=OLAP
+ DUPLICATE KEY(`k1`)
+ COMMENT 'OLAP'
+ DISTRIBUTED BY HASH(`k1`) BUCKETS 10
+ PROPERTIES("replication_num" = "1");
+ """
+ table_names.add(nested_table_dup)
+ // define dup key table with nested table types with two nested scala
+ def nested_table_dup2 = "tbl_csv_array_nested_types_s32"
+ sql "DROP TABLE IF EXISTS ${nested_table_dup2}"
+ sql """
+ CREATE TABLE IF NOT EXISTS ${nested_table_dup2} (
+ `k1` bigint(11) NULL,
+ `c_bool` array<array<boolean>> NULL,
+ `c_tinyint` array<array<tinyint(4)>> NULL,
+ `c_smallint` array<array<smallint(6)>> NULL,
+ `c_int` array<array<int(11)>> NULL,
+ `c_bigint` array<array<bigint(20)>> NULL,
+ `c_largeint` array<array<largeint(40)>> NULL,
+ `c_float` array<array<float>> NULL,
+ `c_double` array<array<double>> NULL,
+ `c_decimal` array<array<decimal(20, 3)>> NULL,
+ `c_decimalv3` array<array<decimalv3(20, 3)>> NULL,
+ `c_date` array<array<date>> NULL,
+ `c_datetime` array<array<datetime>> NULL,
+ `c_datev2` array<array<datev2>> NULL,
+ `c_datetimev2` array<array<datetimev2(0)>> NULL,
+ `c_char` array<array<char(15)>> NULL,
+ `c_varchar` array<array<varchar(100)>> NULL,
+ `c_string` array<array<text>> NULL
+ ) ENGINE=OLAP
+ DUPLICATE KEY(`k1`)
+ COMMENT 'OLAP'
+ DISTRIBUTED BY HASH(`k1`) BUCKETS 10
+ PROPERTIES("replication_num" = "1");
+ """
+ table_names.add(nested_table_dup2)
+
+ // define dup key table with array nested map table types with one nested scala
+ def nested_table_array_map_dup = "tbl_csv_array_map_types_s3"
+ sql "DROP TABLE IF EXISTS ${nested_table_array_map_dup}"
+ sql """
+ CREATE TABLE IF NOT EXISTS ${nested_table_array_map_dup} (
+ `k1` bigint(11) NULL,
+ `c_bool` array<map<boolean, boolean>> NULL,
+ `c_tinyint` array<map<tinyint(4), tinyint(4)>> NULL,
+ `c_smallint` array<map<smallint(6), smallint(6)>> NULL,
+ `c_int` array<map<int(11), int(11)>> NULL,
+ `c_bigint` array<map<bigint(20), bigint(20)>> NULL,
+ `c_largeint` array<map<largeint(40), largeint(40)>> NULL,
+ `c_float` array<map<float, float>> NULL,
+ `c_double` array<map<double, double>> NULL,
+ `c_decimal` array<map<decimal(20, 3), decimal(20, 3)>> NULL,
+ `c_decimalv3` array<map<decimalv3(20, 3), decimalv3(20, 3)>> NULL,
+ `c_date` array<map<date, date>> NULL,
+ `c_datetime` array<map<datetime, datetime>> NULL,
+ `c_datev2` array<map<datev2, datev2>> NULL,
+ `c_datetimev2` array<map<datetimev2(0), datetimev2(0)>> NULL,
+ `c_char` array<map<char(15), char(15)>> NULL,
+ `c_varchar` array<map<varchar(100), varchar(100)>> NULL,
+ `c_string` array<map<text, text>> NULL
+ ) ENGINE=OLAP
+ DUPLICATE KEY(`k1`)
+ COMMENT 'OLAP'
+ DISTRIBUTED BY HASH(`k1`) BUCKETS 10
+ PROPERTIES("replication_num" = "1");
+ """
+ table_names.add(nested_table_array_map_dup)
+
+ // define dup key table with map types with one nested scala
+ def nested_table_map_dup = "tbl_csv_map_types_s3"
+ sql "DROP TABLE IF EXISTS ${nested_table_map_dup}"
+ sql """
+ CREATE TABLE IF NOT EXISTS ${nested_table_map_dup} (
+ `k1` bigint(11) NULL,
+ `c_bool` map<boolean, boolean> NULL,
+ `c_tinyint` map<tinyint(4), tinyint(4)> NULL,
+ `c_smallint` map<smallint(6), smallint(6)> NULL,
+ `c_int` map<int(11), int(11)> NULL,
+ `c_bigint` map<bigint(20), bigint(20)> NULL,
+ `c_largeint` map<largeint(40), largeint(40)> NULL,
+ `c_float` map<float, float> NULL,
+ `c_double` map<double, double> NULL,
+ `c_decimal` map<decimal(20, 3), decimal(20, 3)> NULL,
+ `c_decimalv3` map<decimalv3(20, 3), decimalv3(20, 3)> NULL,
+ `c_date` map<date, date> NULL,
+ `c_datetime` map<datetime, datetime> NULL,
+ `c_datev2` map<datev2, datev2> NULL,
+ `c_datetimev2` map<datetimev2(0), datetimev2(0)> NULL,
+ `c_char` map<char(15), char(15)> NULL,
+ `c_varchar` map<varchar(100), varchar(100)> NULL,
+ `c_string` map<text, text> NULL
+ ) ENGINE=OLAP
+ DUPLICATE KEY(`k1`)
+ COMMENT 'OLAP'
+ DISTRIBUTED BY HASH(`k1`) BUCKETS 10
+ PROPERTIES("replication_num" = "1");
+ """
+ table_names.add(nested_table_map_dup)
+
+ // define dup key table with map nested value array table types with one nested scala
+ def nested_table_map_array_dup = "tbl_csv_map_array_types_s3"
+ sql "DROP TABLE IF EXISTS ${nested_table_map_array_dup}"
+ sql """
+ CREATE TABLE IF NOT EXISTS ${nested_table_map_array_dup} (
+ `k1` bigint(11) NULL,
+ `c_bool` map<boolean, array<boolean>> NULL,
+ `c_tinyint` map<tinyint(4), array<tinyint(4)>> NULL,
+ `c_smallint` map<smallint(6), array<smallint(6)>> NULL,
+ `c_int` map<int(11), array<int(11)>> NULL,
+ `c_bigint` map<bigint(20), array<bigint(20)>> NULL,
+ `c_largeint` map<largeint(40), array<largeint(40)>> NULL,
+ `c_float` map<float, array<float>> NULL,
+ `c_double` map<double, array<double>> NULL,
+ `c_decimal` map<decimal(20, 3), array<decimal(20, 3)>> NULL,
+ `c_decimalv3` map<decimalv3(20, 3), array<decimalv3(20, 3)>> NULL,
+ `c_date` map<date, array<date>> NULL,
+ `c_datetime` map<datetime, array<datetime>> NULL,
+ `c_datev2` map<datev2, array<datev2>> NULL,
+ `c_datetimev2` map<datetimev2(0), array<datetimev2(0)>> NULL,
+ `c_char` map<char(15), array<char(15)>> NULL,
+ `c_varchar` map<varchar(100), array<varchar(100)>> NULL,
+ `c_string` map<text, array<text>> NULL
+ ) ENGINE=OLAP
+ DUPLICATE KEY(`k1`)
+ COMMENT 'OLAP'
+ DISTRIBUTED BY HASH(`k1`) BUCKETS 10
+ PROPERTIES("replication_num" = "1");
+ """
+ table_names.add(nested_table_map_array_dup)
+
+ // step1. select * from s3 with 0-100 items
+ // step2. insert into doris table
+ // step2. query and check
+
+ for (int i = 0; i < 3; ++i) {
Review Comment:
3 -> table_names.size()
--
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] [case](regresscases) add regress cases for nested type nested type with csv format [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #25355:
URL: https://github.com/apache/doris/pull/25355#issuecomment-1761298937
(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.63 seconds
stream load tsv: 551 seconds loaded 74807831229 Bytes, about 129 MB/s
stream load json: 21 seconds loaded 2358488459 Bytes, about 107 MB/s
stream load orc: 64 seconds loaded 1101869774 Bytes, about 16 MB/s
stream load parquet: 33 seconds loaded 861443392 Bytes, about 24 MB/s
insert into select: 28.9 seconds inserted 10000000 Rows, about 346K ops/s
storage size: 17162244213 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