[GitHub] [incubator-doris] yiguolei commented on a diff in pull request #10016: [enhancement]Cache parsed page data for bitshuffle encoding

2022-06-08 Thread GitBox
yiguolei commented on code in PR #10016: URL: https://github.com/apache/incubator-doris/pull/10016#discussion_r893055237 ## be/src/olap/rowset/segment_v2/parsed_page.h: ## @@ -67,17 +70,38 @@ struct ParsedPage { return Status::OK(); } -~ParsedPage() { -

[GitHub] [incubator-doris] yiguolei commented on a diff in pull request #10016: [enhancement]Cache parsed page data for bitshuffle encoding

2022-06-08 Thread GitBox
yiguolei commented on code in PR #10016: URL: https://github.com/apache/incubator-doris/pull/10016#discussion_r893055475 ## be/src/olap/rowset/segment_v2/parsed_page.h: ## @@ -67,17 +70,38 @@ struct ParsedPage { return Status::OK(); } -~ParsedPage() { -

[GitHub] [incubator-doris] lide-reed opened a new issue, #10023: [Bug] One BE instance crashed when executing query with enable_vectorized_engine=true

2022-06-08 Thread GitBox
lide-reed opened a new issue, #10023: URL: https://github.com/apache/incubator-doris/issues/10023 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/incubator-doris/issues?q=is%3Aissue) and found no similar issues. ### Version 1.

[GitHub] [incubator-doris] morningman commented on a diff in pull request #10018: [enhancement] support build libhdfs in arm cpus

2022-06-08 Thread GitBox
morningman commented on code in PR #10018: URL: https://github.com/apache/incubator-doris/pull/10018#discussion_r893058406 ## be/src/io/CMakeLists.txt: ## @@ -31,16 +31,21 @@ set(EXEC_FILES s3_writer.cpp hdfs_reader_writer.cpp file_factory.cpp + hdfs_file_r

[GitHub] [incubator-doris] yiguolei commented on a diff in pull request #10016: [enhancement]Cache parsed page data for bitshuffle encoding

2022-06-08 Thread GitBox
yiguolei commented on code in PR #10016: URL: https://github.com/apache/incubator-doris/pull/10016#discussion_r893059900 ## be/src/olap/rowset/segment_v2/column_reader.cpp: ## @@ -659,20 +662,33 @@ Status FileColumnIterator::_read_data_page(const OrdinalPageIndexIterator& iter)

[GitHub] [incubator-doris] lide-reed opened a new issue, #10024: [Bug] "Show databases" appears "ERROR 2006 (HY000): MySQL server has gone away" when login with a non-exist database

2022-06-08 Thread GitBox
lide-reed opened a new issue, #10024: URL: https://github.com/apache/incubator-doris/issues/10024 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/incubator-doris/issues?q=is%3Aissue) and found no similar issues. ### Version de

[GitHub] [incubator-doris] HappenLee commented on a diff in pull request #9056: [feature-wip](array-type) Add array aggregation functions

2022-06-08 Thread GitBox
HappenLee commented on code in PR #9056: URL: https://github.com/apache/incubator-doris/pull/9056#discussion_r893062130 ## be/src/vec/functions/array/function_array_aggregation.cpp: ## @@ -0,0 +1,300 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more c

[GitHub] [incubator-doris] wangshuo128 commented on pull request #10019: [Enhancement](Nereids) generate pattern by operator

2022-06-08 Thread GitBox
wangshuo128 commented on PR #10019: URL: https://github.com/apache/incubator-doris/pull/10019#issuecomment-1150666122 LGTM -- 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.

[GitHub] [incubator-doris] lide-reed opened a new issue, #10025: [Enhancement] when a csv file include a separator at the end of line, the stream load could ignore it.

2022-06-08 Thread GitBox
lide-reed opened a new issue, #10025: URL: https://github.com/apache/incubator-doris/issues/10025 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/incubator-doris/issues?q=is%3Aissue) and found no similar issues. ### Description

[GitHub] [incubator-doris] Kikyou1997 commented on a diff in pull request #9993: [feature](nereids) Plan Translator

2022-06-08 Thread GitBox
Kikyou1997 commented on code in PR #9993: URL: https://github.com/apache/incubator-doris/pull/9993#discussion_r893072065 ## fe/fe-core/src/main/java/org/apache/doris/nereids/PlanOperatorVisitor.java: ## @@ -0,0 +1,57 @@ +// Licensed to the Apache Software Foundation (ASF) under

[GitHub] [incubator-doris] Kikyou1997 commented on a diff in pull request #9993: [feature](nereids) Plan Translator

2022-06-08 Thread GitBox
Kikyou1997 commented on code in PR #9993: URL: https://github.com/apache/incubator-doris/pull/9993#discussion_r893072238 ## fe/fe-core/src/main/java/org/apache/doris/nereids/PlanOperatorVisitor.java: ## @@ -0,0 +1,57 @@ +// Licensed to the Apache Software Foundation (ASF) under

[GitHub] [incubator-doris] Kikyou1997 commented on a diff in pull request #9993: [feature](nereids) Plan Translator

2022-06-08 Thread GitBox
Kikyou1997 commented on code in PR #9993: URL: https://github.com/apache/incubator-doris/pull/9993#discussion_r893072339 ## fe/fe-core/src/main/java/org/apache/doris/nereids/PlanOperatorVisitor.java: ## @@ -0,0 +1,57 @@ +// Licensed to the Apache Software Foundation (ASF) under

[GitHub] [incubator-doris] kuolei opened a new pull request, #10026: Merge pull request #1 from apache/master

2022-06-08 Thread GitBox
kuolei opened a new pull request, #10026: URL: https://github.com/apache/incubator-doris/pull/10026 merge new code from apache-dors # Proposed changes Issue Number: close #xxx ## Problem Summary: Describe the overview of changes. ## Checklist(Required)

[GitHub] [incubator-doris] kuolei closed pull request #10026: Merge pull request #1 from apache/master

2022-06-08 Thread GitBox
kuolei closed pull request #10026: Merge pull request #1 from apache/master URL: https://github.com/apache/incubator-doris/pull/10026 -- 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 comm

[GitHub] [incubator-doris] EmmyMiao87 commented on pull request #9995: [fix](fe) select stmt will make BE coredump when its castExpr is like cast(int as array<>)

2022-06-08 Thread GitBox
EmmyMiao87 commented on PR #9995: URL: https://github.com/apache/incubator-doris/pull/9995#issuecomment-1150705405 Please add some P0 test -- 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 spe

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9995: [fix](fe) select stmt will make BE coredump when its castExpr is like cast(int as array<>)

2022-06-08 Thread GitBox
github-actions[bot] commented on PR #9995: URL: https://github.com/apache/incubator-doris/pull/9995#issuecomment-1150705782 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 us

[GitHub] [incubator-doris] caoliang-web closed pull request #9944: [doc]Optimize cluster upgrade and Modify binlog load document modification

2022-06-08 Thread GitBox
caoliang-web closed pull request #9944: [doc]Optimize cluster upgrade and Modify binlog load document modification URL: https://github.com/apache/incubator-doris/pull/9944 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [incubator-doris] caoliang-web closed pull request #10012: [doc]restore adds common errors

2022-06-08 Thread GitBox
caoliang-web closed pull request #10012: [doc]restore adds common errors URL: https://github.com/apache/incubator-doris/pull/10012 -- 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

[GitHub] [incubator-doris] zy-kkk closed pull request #9994: [doc]add create table statement for create-materialized-view doc

2022-06-08 Thread GitBox
zy-kkk closed pull request #9994: [doc]add create table statement for create-materialized-view doc URL: https://github.com/apache/incubator-doris/pull/9994 -- 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

[GitHub] [incubator-doris] caoliang-web opened a new pull request, #10027: [doc]Optimize cluster upgrade and binlog load document modification,restor…

2022-06-08 Thread GitBox
caoliang-web opened a new pull request, #10027: URL: https://github.com/apache/incubator-doris/pull/10027 ## Problem Summary: 1.restore adds common errors 2.Optimize cluster upgrade documentation 3.Modify binlog documentation ## Checklist(Required) 1. Does it affect

[GitHub] [incubator-doris] Jibing-Li opened a new pull request, #10028: WIP! NO NOT MERGE! Add HMS external data source.

2022-06-08 Thread GitBox
Jibing-Li opened a new pull request, #10028: URL: https://github.com/apache/incubator-doris/pull/10028 # Proposed changes Implement Hive meta store external data source related classes. Including HMSExternalDataSource, HMSExternalDatabase, HMSExternalTable and so on. Issue Numb

[GitHub] [incubator-doris] zy-kkk opened a new pull request, #10029: [doc]add create table statement for create-materialized-view doc

2022-06-08 Thread GitBox
zy-kkk opened a new pull request, #10029: URL: https://github.com/apache/incubator-doris/pull/10029 # Proposed changes Issue Number: close #xxx ## Problem Summary: Describe the overview of changes. ## Checklist(Required) 1. Does it affect the original behavi

[GitHub] [incubator-doris] englefly opened a new pull request, #10030: [fix] fix q17 rewrite error

2022-06-08 Thread GitBox
englefly opened a new pull request, #10030: URL: https://github.com/apache/incubator-doris/pull/10030 # Proposed changes the current tpch q17 rewrite is not equivalent to original q17. it missed one condition Issue Number: close #xxx ## Problem Summary: Describe the o

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9995: [fix](fe) select stmt will make BE coredump when its castExpr is like cast(int as array<>)

2022-06-08 Thread GitBox
github-actions[bot] commented on PR #9995: URL: https://github.com/apache/incubator-doris/pull/9995#issuecomment-1150705757 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

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #10030: [fix] fix q17 rewrite error

2022-06-08 Thread GitBox
github-actions[bot] commented on PR #10030: URL: https://github.com/apache/incubator-doris/pull/10030#issuecomment-1150725422 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

[GitHub] [incubator-doris] github-actions[bot] commented on pull request #10030: [fix] fix q17 rewrite error

2022-06-08 Thread GitBox
github-actions[bot] commented on PR #10030: URL: https://github.com/apache/incubator-doris/pull/10030#issuecomment-1150725444 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

[GitHub] [incubator-doris] zxealous commented on pull request #10017: [style] Fix the code style defined by class member variables in query…

2022-06-08 Thread GitBox
zxealous commented on PR #10017: URL: https://github.com/apache/incubator-doris/pull/10017#issuecomment-1150728132 @morningman Can this pr be merged? -- 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

[GitHub] [incubator-doris] whutpencil opened a new issue, #10032: [Bug] Brokerload judges the orc empty file logic is defective

2022-06-08 Thread GitBox
whutpencil opened a new issue, #10032: URL: https://github.com/apache/incubator-doris/issues/10032 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/incubator-doris/issues?q=is%3Aissue) and found no similar issues. ### Version 0

<    1   2