[GitHub] [incubator-doris] chaoyli commented on a change in pull request #1200: Merge refactor code into master

chaoyli commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288116269 ## File path: be/src/olap/schema_change.cpp ## @@ -568,20 +546,16 @@ bool RowBlockMerger::merge( // That

[GitHub] [incubator-doris] chaoyli commented on a change in pull request #1200: Merge refactor code into master

chaoyli commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288377185 ## File path: be/src/olap/data_dir.cpp ## @@ -0,0 +1,1011 @@ +// Licensed to the Apache Software Foundation (

[GitHub] [incubator-doris] chaoyli commented on a change in pull request #1200: Merge refactor code into master

chaoyli commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288377246 ## File path: be/src/olap/data_dir.cpp ## @@ -0,0 +1,1011 @@ +// Licensed to the Apache Software Foundation (

[GitHub] [incubator-doris] chaoyli commented on a change in pull request #1200: Merge refactor code into master

chaoyli commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288107491 ## File path: be/src/olap/merger.cpp ## @@ -35,91 +33,70 @@ using std::vector; namespace doris { -Merger

[GitHub] [incubator-doris] yiguolei opened a new pull request #1217: Remove snapshot version in clone copy since it is useless

yiguolei opened a new pull request #1217: Remove snapshot version in clone copy since it is useless URL: https://github.com/apache/incubator-doris/pull/1217 This is an automated message from the Apache Git Service. To respon

[GitHub] [incubator-doris] chaoyli merged pull request #1217: Remove snapshot version in clone copy since it is useless

chaoyli merged pull request #1217: Remove snapshot version in clone copy since it is useless URL: https://github.com/apache/incubator-doris/pull/1217 This is an automated message from the Apache Git Service. To respond to th

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288433748 ## File path: be/src/agent/agent_server.cpp ## @@ -430,10 +416,10 @@ void AgentServer::make_snapshot(TA

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288473292 ## File path: be/src/olap/data_dir.cpp ## @@ -0,0 +1,1011 @@ +// Licensed to the Apache Software Founda

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288457847 ## File path: be/src/agent/task_worker_pool.cpp ## @@ -2254,25 +1638,22 @@ AgentStatus TaskWorkerPool::

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288465340 ## File path: be/src/olap/cumulative_compaction.h ## @@ -102,21 +103,21 @@ class CumulativeCompaction {

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288473111 ## File path: be/src/olap/data_dir.cpp ## @@ -0,0 +1,1011 @@ +// Licensed to the Apache Software Founda

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288477460 ## File path: be/src/olap/olap_cond.cpp ## @@ -555,30 +556,30 @@ bool CondColumn::eval(const BloomFilte

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288459082 ## File path: be/src/exec/olap_scanner.cpp ## @@ -80,29 +80,30 @@ Status OlapScanner::_prepare(

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288474815 ## File path: be/src/olap/data_dir.h ## @@ -21,70 +21,97 @@ #include #include #include +#include

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288476504 ## File path: be/src/olap/field.cpp ## @@ -28,48 +28,46 @@ using std::string; namespace doris { -F

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288466347 ## File path: be/src/olap/cumulative_compaction.h ## @@ -102,21 +103,21 @@ class CumulativeCompaction {

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288459032 ## File path: be/src/exec/olap_scanner.cpp ## @@ -80,29 +80,30 @@ Status OlapScanner::_prepare(

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288459516 ## File path: be/src/exec/olap_scanner.cpp ## @@ -177,11 +178,11 @@ Status OlapScanner::_init_params(

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288467667 ## File path: be/src/olap/data_dir.cpp ## @@ -0,0 +1,1011 @@ +// Licensed to the Apache Software Founda

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288466702 ## File path: be/src/olap/cumulative_compaction.h ## @@ -133,46 +134,29 @@ class CumulativeCompaction {

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288459151 ## File path: be/src/exec/olap_scanner.cpp ## @@ -80,29 +80,30 @@ Status OlapScanner::_prepare(

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288475002 ## File path: be/src/olap/data_dir.h ## @@ -21,70 +21,97 @@ #include #include #include +#include

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288466521 ## File path: be/src/olap/cumulative_compaction.h ## @@ -133,46 +134,29 @@ class CumulativeCompaction {

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1200: Merge refactor code into master

kangpinghuang commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288458681 ## File path: be/src/exec/olap_scan_node.cpp ## @@ -729,7 +729,7 @@ Status OlapScanNode::normalize_in_

[GitHub] [incubator-doris] morningman commented on a change in pull request #1213: Add spatial func

morningman commented on a change in pull request #1213: Add spatial func URL: https://github.com/apache/incubator-doris/pull/1213#discussion_r288488136 ## File path: docs/documentation/cn/sql-reference/sql-functions/spatial-functions/st_x.md ## @@ -0,0 +1,20 @@ +# `ST_X` +

[GitHub] [incubator-doris] morningman commented on a change in pull request #1213: Add spatial func

morningman commented on a change in pull request #1213: Add spatial func URL: https://github.com/apache/incubator-doris/pull/1213#discussion_r288487976 ## File path: docs/documentation/cn/sql-reference/sql-functions/spatial-functions/st_y.md ## @@ -0,0 +1,20 @@ +# `ST_Y` +

[GitHub] [incubator-doris] morningman commented on a change in pull request #1213: Add spatial func

morningman commented on a change in pull request #1213: Add spatial func URL: https://github.com/apache/incubator-doris/pull/1213#discussion_r288488489 ## File path: docs/documentation/cn/sql-reference/sql-functions/spatial-functions/st_polygon.md ## @@ -0,0 +1,20 @@ +# `S

[GitHub] [incubator-doris] morningman commented on a change in pull request #1213: Add spatial func

morningman commented on a change in pull request #1213: Add spatial func URL: https://github.com/apache/incubator-doris/pull/1213#discussion_r288490423 ## File path: docs/documentation/cn/sql-reference/sql-functions/spatial-functions/st_distance_sphere.md ## @@ -0,0 +1,20

[GitHub] [incubator-doris] morningman commented on a change in pull request #1213: Add spatial func

morningman commented on a change in pull request #1213: Add spatial func URL: https://github.com/apache/incubator-doris/pull/1213#discussion_r288487555 ## File path: thirdparty/build-thirdparty.sh ## @@ -558,6 +558,29 @@ build_arrow() { cp -rf ./uriparser_ep-install/li

[GitHub] [incubator-doris] morningman commented on a change in pull request #1213: Add spatial func

morningman commented on a change in pull request #1213: Add spatial func URL: https://github.com/apache/incubator-doris/pull/1213#discussion_r288491344 ## File path: be/src/geo/geo_functions.cpp ## @@ -0,0 +1,307 @@ +// Licensed to the Apache Software Foundation (ASF) under

[GitHub] [incubator-doris] morningman commented on a change in pull request #1213: Add spatial func

morningman commented on a change in pull request #1213: Add spatial func URL: https://github.com/apache/incubator-doris/pull/1213#discussion_r288488386 ## File path: docs/documentation/cn/sql-reference/sql-functions/spatial-functions/st_y.md ## @@ -0,0 +1,20 @@ +# `ST_Y` +

[GitHub] [incubator-doris] morningman commented on a change in pull request #1213: Add spatial func

morningman commented on a change in pull request #1213: Add spatial func URL: https://github.com/apache/incubator-doris/pull/1213#discussion_r288489740 ## File path: docs/documentation/cn/sql-reference/sql-functions/spatial-functions/st_astext.md ## @@ -0,0 +1,20 @@ +# `ST

[GitHub] [incubator-doris] kangkaisen opened a new issue #1218: Setting global parallel_fragment_exec_instance_num variable doesn't work

kangkaisen opened a new issue #1218: Setting global parallel_fragment_exec_instance_num variable doesn't work URL: https://github.com/apache/incubator-doris/issues/1218 **Describe the bug** 1 `set global parallel_fragment_exec_instance_num = 3` in mysql client 2 restart FE 3 the `p

[GitHub] [incubator-doris] kangkaisen opened a new pull request #1219: Replay GlobalVariable by Annotation

kangkaisen opened a new pull request #1219: Replay GlobalVariable by Annotation URL: https://github.com/apache/incubator-doris/pull/1219 Currently, In `replayGlobalVariable ` we need update variables one by one manually. I think which is not reasonable and error-prone. we could use

[GitHub] [incubator-doris] morningman merged pull request #1219: Replay GlobalVariable by Annotation

morningman merged pull request #1219: Replay GlobalVariable by Annotation URL: https://github.com/apache/incubator-doris/pull/1219 This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [incubator-doris] kangkaisen commented on issue #1219: Replay GlobalVariable by Annotation

kangkaisen commented on issue #1219: Replay GlobalVariable by Annotation URL: https://github.com/apache/incubator-doris/pull/1219#issuecomment-496906415 Fix https://github.com/apache/incubator-doris/issues/1218 This is an auto

[GitHub] [incubator-doris] kangkaisen closed issue #1218: Setting global parallel_fragment_exec_instance_num variable doesn't work

kangkaisen closed issue #1218: Setting global parallel_fragment_exec_instance_num variable doesn't work URL: https://github.com/apache/incubator-doris/issues/1218 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-doris] kangkaisen opened a new issue #1220: VariableMgr flag could be simplified

kangkaisen opened a new issue #1220: VariableMgr flag could be simplified URL: https://github.com/apache/incubator-doris/issues/1220 I think the following three flags could simplify to one flag `GLOBAL_MODIFY_ONLY`, which means the variable could only modify in global level, if a variable

[GitHub] [incubator-doris] qdore opened a new issue #1221: Use multi condition cost much more time to return?

qdore opened a new issue #1221: Use multi condition cost much more time to return? URL: https://github.com/apache/incubator-doris/issues/1221 **Describe the bug** A clear and concise description of what the bug is. ![image](https://user-images.githubusercontent.com/6831829/58563786-5

[GitHub] [incubator-doris] imay commented on issue #1221: Use multi condition cost much more time to return?

imay commented on issue #1221: Use multi condition cost much more time to return? URL: https://github.com/apache/incubator-doris/issues/1221#issuecomment-496956701 It is bad for Doris to handle OR predicate. For your query, Doris would read all data and do OR operation. It is bad when eve

[GitHub] [incubator-doris] EmmyMiao87 opened a new pull request #1222: Fix olap engine bug of decimal agg

EmmyMiao87 opened a new pull request #1222: Fix olap engine bug of decimal agg URL: https://github.com/apache/incubator-doris/pull/1222 This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [incubator-doris] kangkaisen opened a new issue #1223: NPE when create colocate table

kangkaisen opened a new issue #1223: NPE when create colocate table URL: https://github.com/apache/incubator-doris/issues/1223 **Describe the bug** ``` Stacktrace was: java.lang.NullPointerException at org.apache.doris.catalog.Catalog.createOlapTable(Catalog.java:3599) at org

[GitHub] [incubator-doris] kangkaisen opened a new pull request #1224: Check colocate table name should be case-sensitive

kangkaisen opened a new pull request #1224: Check colocate table name should be case-sensitive URL: https://github.com/apache/incubator-doris/pull/1224 Fix https://github.com/apache/incubator-doris/issues/1223 This is an auto

[GitHub] [incubator-doris] EmmyMiao87 opened a new issue #1225: The error result of decimal agg

EmmyMiao87 opened a new issue #1225: The error result of decimal agg URL: https://github.com/apache/incubator-doris/issues/1225 **Describe the bug** The result of decimal agg is error when stream load. **To Reproduce** Steps to reproduce the behavior: 1. make a file with two

[GitHub] [incubator-doris] EmmyMiao87 opened a new pull request #1226: [ISSUE-1225] Fix the bug of += decimal in olap engine

EmmyMiao87 opened a new pull request #1226: [ISSUE-1225] Fix the bug of += decimal in olap engine URL: https://github.com/apache/incubator-doris/pull/1226 This change fix the olap engine bug of decimal agg. Using ^ instead of * to judge result is less then zero. The result of * will be

[GitHub] [incubator-doris] chaoyli closed pull request #1200: Merge refactor code into master

chaoyli closed pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-doris] imay closed pull request #1224: Check colocate table name should be case-sensitive

imay closed pull request #1224: Check colocate table name should be case-sensitive URL: https://github.com/apache/incubator-doris/pull/1224 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-doris] kangkaisen opened a new pull request #1224: Check colocate table name should be case-sensitive

kangkaisen opened a new pull request #1224: Check colocate table name should be case-sensitive URL: https://github.com/apache/incubator-doris/pull/1224 Fix https://github.com/apache/incubator-doris/issues/1223 This is an auto

[GitHub] [incubator-doris] chaoyli opened a new pull request #1227: Add negative load from StreamLoad

chaoyli opened a new pull request #1227: Add negative load from StreamLoad URL: https://github.com/apache/incubator-doris/pull/1227 This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] [incubator-doris] morningman opened a new pull request #1228: Modify some thirdparties

morningman opened a new pull request #1228: Modify some thirdparties URL: https://github.com/apache/incubator-doris/pull/1228 1. Change Kafka java client from 2.0.0 to 0.10.1.1. Because high version client may not support low server server. 2. Enable SSL in librdkafka --

[GitHub] [incubator-doris] chaoyli opened a new pull request #1229: Initialize member variable in Tablet

chaoyli opened a new pull request #1229: Initialize member variable in Tablet URL: https://github.com/apache/incubator-doris/pull/1229 This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [incubator-doris] imay merged pull request #1228: Modify some thirdparties

imay merged pull request #1228: Modify some thirdparties URL: https://github.com/apache/incubator-doris/pull/1228 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[GitHub] [incubator-doris] imay commented on a change in pull request #1226: Fix the bug of += decimal in olap engine

imay commented on a change in pull request #1226: Fix the bug of += decimal in olap engine URL: https://github.com/apache/incubator-doris/pull/1226#discussion_r288988565 ## File path: be/test/olap/field_info_test.cpp ## @@ -0,0 +1,26 @@ +#include Review comment: lic

[GitHub] [incubator-doris] imay closed pull request #1227: Add negative load from StreamLoad

imay closed pull request #1227: Add negative load from StreamLoad URL: https://github.com/apache/incubator-doris/pull/1227 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-doris] chaoyli opened a new pull request #1227: Add negative load from StreamLoad

chaoyli opened a new pull request #1227: Add negative load from StreamLoad URL: https://github.com/apache/incubator-doris/pull/1227 This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] [incubator-doris] imay closed pull request #1224: Check colocate table name should be case-sensitive

imay closed pull request #1224: Check colocate table name should be case-sensitive URL: https://github.com/apache/incubator-doris/pull/1224 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-doris] kangkaisen opened a new pull request #1224: Check colocate table name should be case-sensitive

kangkaisen opened a new pull request #1224: Check colocate table name should be case-sensitive URL: https://github.com/apache/incubator-doris/pull/1224 Fix https://github.com/apache/incubator-doris/issues/1223 This is an auto

[GitHub] [incubator-doris] imay commented on a change in pull request #1226: Fix the bug of += decimal in olap engine

imay commented on a change in pull request #1226: Fix the bug of += decimal in olap engine URL: https://github.com/apache/incubator-doris/pull/1226#discussion_r288991677 ## File path: be/test/olap/field_info_test.cpp ## @@ -0,0 +1,43 @@ +// Licensed to the Apache Software

[GitHub] [incubator-doris] imay closed pull request #1227: Add negative load from StreamLoad

imay closed pull request #1227: Add negative load from StreamLoad URL: https://github.com/apache/incubator-doris/pull/1227 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-doris] chaoyli opened a new pull request #1227: Add negative load from StreamLoad

chaoyli opened a new pull request #1227: Add negative load from StreamLoad URL: https://github.com/apache/incubator-doris/pull/1227 This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] [incubator-doris] imay closed issue #1223: NPE when create colocate table

imay closed issue #1223: NPE when create colocate table URL: https://github.com/apache/incubator-doris/issues/1223 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [incubator-doris] imay merged pull request #1224: Check colocate table name should be case-sensitive

imay merged pull request #1224: Check colocate table name should be case-sensitive URL: https://github.com/apache/incubator-doris/pull/1224 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-doris] imay merged pull request #1226: Fix the bug of += decimal in olap engine

imay merged pull request #1226: Fix the bug of += decimal in olap engine URL: https://github.com/apache/incubator-doris/pull/1226 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-doris] imay merged pull request #1227: Add negative load from StreamLoad

imay merged pull request #1227: Add negative load from StreamLoad URL: https://github.com/apache/incubator-doris/pull/1227 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-doris] chaoyli merged pull request #1229: Initialize member variable in Tablet

chaoyli merged pull request #1229: Initialize member variable in Tablet URL: https://github.com/apache/incubator-doris/pull/1229 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-doris] chaoyli opened a new pull request #1200: Merge refactor code into master

chaoyli opened a new pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200 1. Refactoring be is to clarify the structure the codes. 2. Use unique id to indicate a rowset. Nameing rowset with tablet_id and version will lead to

[GitHub] [incubator-doris] imay merged pull request #1213: Add spatial func

imay merged pull request #1213: Add spatial func URL: https://github.com/apache/incubator-doris/pull/1213 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289274992 ## File path: fe/src/main/java/org/apache/doris/qe/MultiLoadMgr.java ## @@ -379,10 +378,14 @@ public D

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289275359 ## File path: fe/src/main/java/org/apache/doris/catalog/BrokerTable.java ## @@ -222,7 +238,7 @@ public

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289273174 ## File path: be/src/exec/broker_reader.cpp ## @@ -112,24 +112,29 @@ Status BrokerReader::open() {

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289274242 ## File path: be/src/exec/parquet_scanner.cpp ## @@ -0,0 +1,277 @@ +// Licensed to the Apache Software

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289274081 ## File path: be/src/exec/parquet_reader.cpp ## @@ -0,0 +1,461 @@ +// Licensed to the Apache Software

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289273867 ## File path: be/src/exec/local_file_reader.cpp ## @@ -37,52 +39,78 @@ Status LocalFileReader::open()

[GitHub] [incubator-doris] KevinLinGit opened a new pull request #1230: feat: release udf headers & lib

KevinLinGit opened a new pull request #1230: feat: release udf headers & lib URL: https://github.com/apache/incubator-doris/pull/1230 remove internal headers from udf.h release udf headers & lib This is an automated message

[GitHub] [incubator-doris] KevinLinGit closed pull request #1230: feat: release udf headers & lib

KevinLinGit closed pull request #1230: feat: release udf headers & lib URL: https://github.com/apache/incubator-doris/pull/1230 This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [incubator-doris] morningman closed pull request #1222: Fix olap engine bug of decimal agg

morningman closed pull request #1222: Fix olap engine bug of decimal agg URL: https://github.com/apache/incubator-doris/pull/1222 This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [incubator-doris] KevinLinGit opened a new pull request #1231: feat: release udf headers & lib

KevinLinGit opened a new pull request #1231: feat: release udf headers & lib URL: https://github.com/apache/incubator-doris/pull/1231 add fPIC to libDorisUdf.a add StringVal::resize remove internal headers release udf headers & lib --

[GitHub] [incubator-doris] imay commented on a change in pull request #1231: feat: release udf headers & lib

imay commented on a change in pull request #1231: feat: release udf headers & lib URL: https://github.com/apache/incubator-doris/pull/1231#discussion_r289281817 ## File path: be/src/udf/udf.h ## @@ -21,8 +21,8 @@ #include #include -#include "common/logging.h" -#inclu

[GitHub] [incubator-doris] worker24h commented on a change in pull request #1173: load data from Parquet file (Issues #911)

worker24h commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289286730 ## File path: fe/src/main/java/org/apache/doris/catalog/BrokerTable.java ## @@ -222,7 +238,7 @@ p

[GitHub] [incubator-doris] imay merged pull request #1231: feat: release udf headers & lib

imay merged pull request #1231: feat: release udf headers & lib URL: https://github.com/apache/incubator-doris/pull/1231 This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [incubator-doris] chenhao7253886 closed pull request #1210: Add optimization with cost and fix unit test

chenhao7253886 closed pull request #1210: Add optimization with cost and fix unit test URL: https://github.com/apache/incubator-doris/pull/1210 This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [incubator-doris] imay opened a new pull request #1232: Fix FragmentMgrTest crash sometimes

imay opened a new pull request #1232: Fix FragmentMgrTest crash sometimes URL: https://github.com/apache/incubator-doris/pull/1232 This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [incubator-doris] morningman merged pull request #1232: Fix FragmentMgrTest crash sometimes

morningman merged pull request #1232: Fix FragmentMgrTest crash sometimes URL: https://github.com/apache/incubator-doris/pull/1232 This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [incubator-doris] chenhao7253886 opened a new pull request #1233: Add optimization with cost and fix unit test

chenhao7253886 opened a new pull request #1233: Add optimization with cost and fix unit test URL: https://github.com/apache/incubator-doris/pull/1233 This is an automated message from the Apache Git Service. To respond to th

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289645236 ## File path: be/src/exec/parquet_reader.h ## @@ -0,0 +1,105 @@ +// Licensed to the Apache Software Fo

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289645240 ## File path: be/src/exec/parquet_reader.h ## @@ -0,0 +1,105 @@ +// Licensed to the Apache Software Fo

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289645219 ## File path: be/src/exec/parquet_reader.h ## @@ -0,0 +1,105 @@ +// Licensed to the Apache Software Fo

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289644426 ## File path: be/src/exec/parquet_reader.cpp ## @@ -0,0 +1,461 @@ +// Licensed to the Apache Software

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289645244 ## File path: be/src/exec/parquet_reader.h ## @@ -0,0 +1,105 @@ +// Licensed to the Apache Software Fo

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289645248 ## File path: be/src/exec/parquet_reader.h ## @@ -0,0 +1,105 @@ +// Licensed to the Apache Software Fo

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289643942 ## File path: be/src/exec/local_file_reader.cpp ## @@ -14,13 +14,15 @@ // KIND, either express or imp

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289645180 ## File path: be/src/exec/parquet_reader.cpp ## @@ -0,0 +1,461 @@ +// Licensed to the Apache Software

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289643142 ## File path: fe/src/main/java/org/apache/doris/catalog/BrokerTable.java ## @@ -202,7 +218,7 @@ public

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289645794 ## File path: be/src/exec/parquet_reader.cpp ## @@ -0,0 +1,461 @@ +// Licensed to the Apache Software

[GitHub] [incubator-doris] imay commented on a change in pull request #1173: load data from Parquet file (Issues #911)

imay commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289645385 ## File path: be/src/exec/parquet_reader.cpp ## @@ -0,0 +1,461 @@ +// Licensed to the Apache Software

[GitHub] [incubator-doris] chaoyli closed pull request #1200: Merge refactor code into master

chaoyli closed pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-doris] worker24h commented on a change in pull request #1173: load data from Parquet file (Issues #911)

worker24h commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289669282 ## File path: be/src/exec/parquet_reader.cpp ## @@ -0,0 +1,461 @@ +// Licensed to the Apache Soft

[GitHub] [incubator-doris] worker24h commented on a change in pull request #1173: load data from Parquet file (Issues #911)

worker24h commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289669300 ## File path: be/src/exec/parquet_reader.h ## @@ -0,0 +1,105 @@ +// Licensed to the Apache Softwa

[GitHub] [incubator-doris] worker24h commented on a change in pull request #1173: load data from Parquet file (Issues #911)

worker24h commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289669288 ## File path: be/src/exec/parquet_reader.h ## @@ -0,0 +1,105 @@ +// Licensed to the Apache Softwa

[GitHub] [incubator-doris] worker24h commented on a change in pull request #1173: load data from Parquet file (Issues #911)

worker24h commented on a change in pull request #1173: load data from Parquet file (Issues #911) URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r289669322 ## File path: be/src/exec/parquet_reader.h ## @@ -0,0 +1,105 @@ +// Licensed to the Apache Softwa

<    1   2   3   4   5   6   7   8   9   10   >