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

2019-05-28 Thread GitBox
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

2019-05-28 Thread GitBox
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

2019-05-28 Thread GitBox
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] chaoyli commented on a change in pull request #1200: Merge refactor code into master

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

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

2019-05-28 Thread GitBox
chaoyli commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288398316 ## File path: be/src/olap/task/engine_clone_task.cpp ## @@ -0,0 +1,903 @@ +// Licensed to the Apache Software

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

2019-05-28 Thread GitBox
chaoyli commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288380781 ## File path: be/src/olap/olap_snapshot_converter.cpp ## @@ -0,0 +1,531 @@ +// Licensed to the Apache Softwar

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

2019-05-28 Thread GitBox
chaoyli commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288398752 ## File path: be/src/olap/task/engine_clone_task.cpp ## @@ -0,0 +1,903 @@ +// Licensed to the Apache Software

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

2019-05-28 Thread GitBox
chaoyli commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288378703 ## 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

2019-05-28 Thread GitBox
chaoyli commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288394171 ## 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

2019-05-28 Thread GitBox
chaoyli commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288380282 ## File path: be/src/olap/olap_snapshot_converter.cpp ## @@ -0,0 +1,531 @@ +// Licensed to the Apache Softwar

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

2019-05-28 Thread GitBox
chaoyli commented on a change in pull request #1200: Merge refactor code into master URL: https://github.com/apache/incubator-doris/pull/1200#discussion_r288381230 ## File path: be/src/olap/olap_snapshot_converter.cpp ## @@ -0,0 +1,531 @@ +// Licensed to the Apache Softwar

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

2019-05-28 Thread GitBox
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] kangkaisen commented on issue #1214: cannot find symbol when compile FE with JDK 12

2019-05-28 Thread GitBox
kangkaisen commented on issue #1214: cannot find symbol when compile FE with JDK 12 URL: https://github.com/apache/incubator-doris/issues/1214#issuecomment-496795347 OK. I could do this work, but I am busy recently. I will do this work when I have time. Before I do it, I will discus

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

2019-05-28 Thread GitBox
imay closed 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 opened a new pull request #1213: Add spatial func

2019-05-28 Thread GitBox
imay opened a new 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

[GitHub] [incubator-doris] imay closed issue #1214: cannot find symbol when compile FE with JDK 12

2019-05-28 Thread GitBox
imay closed issue #1214: cannot find symbol when compile FE with JDK 12 URL: https://github.com/apache/incubator-doris/issues/1214 This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [incubator-doris] imay merged pull request #1215: Remove unnecessary import sun.security.ssl.Debug

2019-05-28 Thread GitBox
imay merged pull request #1215: Remove unnecessary import sun.security.ssl.Debug URL: https://github.com/apache/incubator-doris/pull/1215 This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [incubator-doris] imay commented on issue #1214: cannot find symbol when compile FE with JDK 12

2019-05-28 Thread GitBox
imay commented on issue #1214: cannot find symbol when compile FE with JDK 12 URL: https://github.com/apache/incubator-doris/issues/1214#issuecomment-496780110 > Besides, **I think we should add a [maven-checkstyle-plugin](https://maven.apache.org/plugins/maven-checkstyle-plugin/) to chec

[GitHub] [incubator-doris] kangkaisen opened a new issue #1216: Doris Stream Load API response has changed in Doris 0.10

2019-05-28 Thread GitBox
kangkaisen opened a new issue #1216: Doris Stream Load API response has changed in Doris 0.10 URL: https://github.com/apache/incubator-doris/issues/1216 The old response is : ``` { "Status": "Success", "Message": "OK", "NumberLoadedRows": 2, "NumberFiltered

[GitHub] [incubator-doris] kangkaisen closed issue #1216: Doris Stream Load API response has changed in Doris 0.10

2019-05-28 Thread GitBox
kangkaisen closed issue #1216: Doris Stream Load API response has changed in Doris 0.10 URL: https://github.com/apache/incubator-doris/issues/1216 This is an automated message from the Apache Git Service. To respond to the m

[GitHub] [incubator-doris] kangkaisen opened a new pull request #1215: Remove unnecessary import sun.security.ssl.Debug

2019-05-28 Thread GitBox
kangkaisen opened a new pull request #1215: Remove unnecessary import sun.security.ssl.Debug URL: https://github.com/apache/incubator-doris/pull/1215 Fix https://github.com/apache/incubator-doris/issues/1214 This is an autom

[GitHub] [incubator-doris] kangkaisen opened a new issue #1214: cannot find symbol when compile FE with JDK 12

2019-05-28 Thread GitBox
kangkaisen opened a new issue #1214: cannot find symbol when compile FE with JDK 12 URL: https://github.com/apache/incubator-doris/issues/1214 **Describe the bug** The following error will occur when compile FE with JDK 12: ``` [ERROR] /home/kangkaisen/palo/fe/src/main/java/org/ap

[GitHub] [incubator-doris] wengad82 commented on issue #1212: bin/start_fe.sh: line 59: /home/doris/run/fe/conf/fe.conf: No such file or directory

2019-05-28 Thread GitBox
wengad82 commented on issue #1212: bin/start_fe.sh: line 59: /home/doris/run/fe/conf/fe.conf: No such file or directory URL: https://github.com/apache/incubator-doris/issues/1212#issuecomment-496771712 yes,theres is a fe.conf under /home/doris/run/fe/conf [root@study conf]# docker e

[GitHub] [incubator-doris] imay opened a new pull request #1213: Add spatial func

2019-05-28 Thread GitBox
imay opened a new 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

[GitHub] [incubator-doris] morningman commented on issue #1212: bin/start_fe.sh: line 59: /home/doris/run/fe/conf/fe.conf: No such file or directory

2019-05-28 Thread GitBox
morningman commented on issue #1212: bin/start_fe.sh: line 59: /home/doris/run/fe/conf/fe.conf: No such file or directory URL: https://github.com/apache/incubator-doris/issues/1212#issuecomment-496769155 You can first attach to the container, and see if there has file: `/home/doris/run/fe

[GitHub] [incubator-doris] morningman closed issue #1201: Too many thrift connection from BE to FE

2019-05-28 Thread GitBox
morningman closed issue #1201: Too many thrift connection from BE to FE URL: https://github.com/apache/incubator-doris/issues/1201 This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [incubator-doris] wengad82 opened a new issue #1212: bin/start_fe.sh: line 59: /home/doris/run/fe/conf/fe.conf: No such file or directory

2019-05-28 Thread GitBox
wengad82 opened a new issue #1212: bin/start_fe.sh: line 59: /home/doris/run/fe/conf/fe.conf: No such file or directory URL: https://github.com/apache/incubator-doris/issues/1212 Dockerfile: > FROM centos:7.2.1511 RUN mkdir /home/doris/run/ -p # copy jdk and doris binar

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1211: Change FileDownloader to http client

2019-05-28 Thread GitBox
kangpinghuang commented on a change in pull request #1211: Change FileDownloader to http client URL: https://github.com/apache/incubator-doris/pull/1211#discussion_r288369590 ## File path: be/src/olap/task/engine_batch_load_task.cpp ## @@ -113,45 +113,31 @@ AgentStatus Eng

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1211: Change FileDownloader to http client

2019-05-28 Thread GitBox
kangpinghuang commented on a change in pull request #1211: Change FileDownloader to http client URL: https://github.com/apache/incubator-doris/pull/1211#discussion_r288370414 ## File path: be/src/olap/task/engine_batch_load_task.cpp ## @@ -218,96 +172,94 @@ void EngineBatc

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1211: Change FileDownloader to http client

2019-05-28 Thread GitBox
kangpinghuang commented on a change in pull request #1211: Change FileDownloader to http client URL: https://github.com/apache/incubator-doris/pull/1211#discussion_r288370375 ## File path: be/src/olap/task/engine_batch_load_task.cpp ## @@ -218,96 +172,94 @@ void EngineBatc

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1211: Change FileDownloader to http client

2019-05-28 Thread GitBox
kangpinghuang commented on a change in pull request #1211: Change FileDownloader to http client URL: https://github.com/apache/incubator-doris/pull/1211#discussion_r288369809 ## File path: be/src/olap/task/engine_batch_load_task.cpp ## @@ -218,96 +172,94 @@ void EngineBatc

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1211: Change FileDownloader to http client

2019-05-28 Thread GitBox
kangpinghuang commented on a change in pull request #1211: Change FileDownloader to http client URL: https://github.com/apache/incubator-doris/pull/1211#discussion_r288369936 ## File path: be/src/olap/task/engine_batch_load_task.cpp ## @@ -218,96 +172,94 @@ void EngineBatc

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1211: Change FileDownloader to http client

2019-05-28 Thread GitBox
kangpinghuang commented on a change in pull request #1211: Change FileDownloader to http client URL: https://github.com/apache/incubator-doris/pull/1211#discussion_r288370468 ## File path: be/src/olap/task/engine_batch_load_task.cpp ## @@ -316,11 +268,9 @@ AgentStatus Engi

[GitHub] [incubator-doris] kangpinghuang commented on a change in pull request #1211: Change FileDownloader to http client

2019-05-28 Thread GitBox
kangpinghuang commented on a change in pull request #1211: Change FileDownloader to http client URL: https://github.com/apache/incubator-doris/pull/1211#discussion_r288370338 ## File path: be/src/olap/task/engine_batch_load_task.cpp ## @@ -218,96 +172,94 @@ void EngineBatc

[GitHub] [incubator-doris] chaoyli merged pull request #1211: Change FileDownloader to http client

2019-05-28 Thread GitBox
chaoyli merged pull request #1211: Change FileDownloader to http client URL: https://github.com/apache/incubator-doris/pull/1211 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-doris] yiguolei opened a new pull request #1211: Change FileDownloader to http client

2019-05-28 Thread GitBox
yiguolei opened a new pull request #1211: Change FileDownloader to http client URL: https://github.com/apache/incubator-doris/pull/1211 This is an automated message from the Apache Git Service. To respond to the message, plea

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

2019-05-28 Thread GitBox
chenhao7253886 opened a new 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 th

[GitHub] [incubator-doris] chaoyli merged pull request #1209: Olapsnapshot converter not set tablet uid correctly

2019-05-28 Thread GitBox
chaoyli merged pull request #1209: Olapsnapshot converter not set tablet uid correctly URL: https://github.com/apache/incubator-doris/pull/1209 This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [incubator-doris] yiguolei opened a new pull request #1209: Olapsnapshot converter not set tablet uid correctly

2019-05-28 Thread GitBox
yiguolei opened a new pull request #1209: Olapsnapshot converter not set tablet uid correctly URL: https://github.com/apache/incubator-doris/pull/1209 This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-doris] imay merged pull request #1208: Fix datediff function

2019-05-28 Thread GitBox
imay merged pull request #1208: Fix datediff function URL: https://github.com/apache/incubator-doris/pull/1208 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and