[GitHub] [incubator-doris] chenhao7253886 opened a new issue #1493: ifnull return wrong type value in aggregation function

2019-07-16 Thread GitBox
chenhao7253886 opened a new issue #1493: ifnull return wrong type value in aggregation function URL: https://github.com/apache/incubator-doris/issues/1493 The following query: mysql> select sum(ifnull(0.0,0)) from baseall'; ERROR 1064 (HY000): SUM requires a numeric parameter: sum(ifn

[GitHub] [incubator-doris] morningman commented on a change in pull request #1488: Add administrator guide of load

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1488: Add administrator guide of load URL: https://github.com/apache/incubator-doris/pull/1488#discussion_r304205161 ## File path: docs/documentation/cn/administrator-guide/load-data/load-manual.md ## @@ -0,0 +1,256 @@ +#

[GitHub] [incubator-doris] morningman commented on a change in pull request #1488: Add administrator guide of load

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1488: Add administrator guide of load URL: https://github.com/apache/incubator-doris/pull/1488#discussion_r304205654 ## File path: docs/documentation/cn/administrator-guide/load-data/load-manual.md ## @@ -0,0 +1,256 @@ +#

[GitHub] [incubator-doris] morningman commented on a change in pull request #1488: Add administrator guide of load

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1488: Add administrator guide of load URL: https://github.com/apache/incubator-doris/pull/1488#discussion_r304206746 ## File path: docs/documentation/cn/administrator-guide/load-data/broker-load-manual.md ## @@ -1,17 +1,8

[GitHub] [incubator-doris] morningman commented on a change in pull request #1488: Add administrator guide of load

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1488: Add administrator guide of load URL: https://github.com/apache/incubator-doris/pull/1488#discussion_r304204555 ## File path: docs/documentation/cn/administrator-guide/load-data/load-manual.md ## @@ -0,0 +1,256 @@ +#

[GitHub] [incubator-doris] morningman commented on a change in pull request #1488: Add administrator guide of load

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1488: Add administrator guide of load URL: https://github.com/apache/incubator-doris/pull/1488#discussion_r304204467 ## File path: docs/documentation/cn/administrator-guide/load-data/load-manual.md ## @@ -0,0 +1,256 @@ +#

[GitHub] [incubator-doris] morningman commented on a change in pull request #1488: Add administrator guide of load

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1488: Add administrator guide of load URL: https://github.com/apache/incubator-doris/pull/1488#discussion_r304204990 ## File path: docs/documentation/cn/administrator-guide/load-data/load-manual.md ## @@ -0,0 +1,256 @@ +#

[GitHub] [incubator-doris] morningman commented on a change in pull request #1488: Add administrator guide of load

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1488: Add administrator guide of load URL: https://github.com/apache/incubator-doris/pull/1488#discussion_r304241005 ## File path: docs/documentation/cn/administrator-guide/load-data/insert-into-manual.md ## @@ -0,0 +1,31

[GitHub] [incubator-doris] morningman commented on a change in pull request #1488: Add administrator guide of load

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1488: Add administrator guide of load URL: https://github.com/apache/incubator-doris/pull/1488#discussion_r304204271 ## File path: docs/documentation/cn/administrator-guide/load-data/load-manual.md ## @@ -0,0 +1,256 @@ +#

[GitHub] [incubator-doris] morningman commented on a change in pull request #1488: Add administrator guide of load

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1488: Add administrator guide of load URL: https://github.com/apache/incubator-doris/pull/1488#discussion_r304205982 ## File path: docs/documentation/cn/administrator-guide/load-data/load-manual.md ## @@ -0,0 +1,256 @@ +#

[GitHub] [incubator-doris] morningman commented on a change in pull request #1488: Add administrator guide of load

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1488: Add administrator guide of load URL: https://github.com/apache/incubator-doris/pull/1488#discussion_r304207411 ## File path: docs/documentation/cn/administrator-guide/load-data/broker-load-manual.md ## @@ -1,17 +1,8

[GitHub] [incubator-doris] morningman commented on a change in pull request #1488: Add administrator guide of load

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1488: Add administrator guide of load URL: https://github.com/apache/incubator-doris/pull/1488#discussion_r304204773 ## File path: docs/documentation/cn/administrator-guide/load-data/load-manual.md ## @@ -0,0 +1,256 @@ +#

[GitHub] [incubator-doris] morningman commented on a change in pull request #1488: Add administrator guide of load

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1488: Add administrator guide of load URL: https://github.com/apache/incubator-doris/pull/1488#discussion_r304205458 ## File path: docs/documentation/cn/administrator-guide/load-data/load-manual.md ## @@ -0,0 +1,256 @@ +#

[GitHub] [incubator-doris] chenhao7253886 opened a new issue #1492: NULL's operator "<=>" get wrong result

2019-07-16 Thread GitBox
chenhao7253886 opened a new issue #1492: NULL's operator "<=>" get wrong result URL: https://github.com/apache/incubator-doris/issues/1492 now , the result of comparing null with other value by operator "<=>" is wrong, for example: mysql> select null <=> null; +-+ | NU

[GitHub] [incubator-doris] morningman merged pull request #1490: Add partition id to tablet meta in be

2019-07-16 Thread GitBox
morningman merged pull request #1490: Add partition id to tablet meta in be URL: https://github.com/apache/incubator-doris/pull/1490 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-doris] imay commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally

2019-07-16 Thread GitBox
imay commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally URL: https://github.com/apache/incubator-doris/pull/1489#discussion_r304231166 ## File path: gensrc/proto/internal_service.proto ## @@ -114,8 +114,16 @@ me

[GitHub] [incubator-doris] chenhao7253886 opened a new pull request #1491: Fix bug that <=> operator get wrong result

2019-07-16 Thread GitBox
chenhao7253886 opened a new pull request #1491: Fix bug that <=> operator get wrong result URL: https://github.com/apache/incubator-doris/pull/1491 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-doris] morningman commented on a change in pull request #1490: Add partition id to tablet meta in be

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1490: Add partition id to tablet meta in be URL: https://github.com/apache/incubator-doris/pull/1490#discussion_r304200443 ## File path: fe/src/main/java/org/apache/doris/task/UpdateTabletMetaInfoTask.java ## @@ -0,0 +1,69

[GitHub] [incubator-doris] morningman commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally URL: https://github.com/apache/incubator-doris/pull/1489#discussion_r304196680 ## File path: be/src/runtime/fragment_mgr.cpp ## @@ -209,9 +209,12 @@

[GitHub] [incubator-doris] morningman commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally URL: https://github.com/apache/incubator-doris/pull/1489#discussion_r304195377 ## File path: fe/src/main/java/org/apache/doris/qe/Coordinator.java ##

[GitHub] [incubator-doris] morningman commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally URL: https://github.com/apache/incubator-doris/pull/1489#discussion_r304195342 ## File path: gensrc/proto/internal_service.proto ## @@ -116,6 +116,12

[GitHub] [incubator-doris] morningman commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally

2019-07-16 Thread GitBox
morningman commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally URL: https://github.com/apache/incubator-doris/pull/1489#discussion_r304195321 ## File path: gensrc/proto/internal_service.proto ## @@ -116,6 +116,12

[GitHub] [incubator-doris] imay commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally

2019-07-16 Thread GitBox
imay commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally URL: https://github.com/apache/incubator-doris/pull/1489#discussion_r304190411 ## File path: fe/src/main/java/org/apache/doris/qe/Coordinator.java ## @@ -1

[GitHub] [incubator-doris] imay commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally

2019-07-16 Thread GitBox
imay commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally URL: https://github.com/apache/incubator-doris/pull/1489#discussion_r304191773 ## File path: fe/src/main/java/org/apache/doris/qe/Coordinator.java ## @@ -6

[GitHub] [incubator-doris] imay commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally

2019-07-16 Thread GitBox
imay commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally URL: https://github.com/apache/incubator-doris/pull/1489#discussion_r304191098 ## File path: be/src/runtime/fragment_mgr.cpp ## @@ -209,9 +209,12 @@ Status

[GitHub] [incubator-doris] imay commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally

2019-07-16 Thread GitBox
imay commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally URL: https://github.com/apache/incubator-doris/pull/1489#discussion_r304191576 ## File path: fe/src/main/java/org/apache/doris/rpc/BackendServiceProxy.java

[GitHub] [incubator-doris] imay commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally

2019-07-16 Thread GitBox
imay commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally URL: https://github.com/apache/incubator-doris/pull/1489#discussion_r304189958 ## File path: gensrc/proto/internal_service.proto ## @@ -116,6 +116,12 @@ me

[GitHub] [incubator-doris] imay commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally

2019-07-16 Thread GitBox
imay commented on a change in pull request #1489: Remove query status report from BE when query is cancelled normally URL: https://github.com/apache/incubator-doris/pull/1489#discussion_r304189361 ## File path: gensrc/proto/internal_service.proto ## @@ -116,6 +116,12 @@ me

[GitHub] [incubator-doris] yiguolei opened a new pull request #1490: Add partition id to tablet meta in be

2019-07-16 Thread GitBox
yiguolei opened a new pull request #1490: Add partition id to tablet meta in be URL: https://github.com/apache/incubator-doris/pull/1490 FE uses partition_id to publish version. BE should check whether all tablets related with this partition have the version. But Tablet in BE does not have

[GitHub] [incubator-doris] morningman opened a new pull request #1489: Remove query status report from BE when query is cancelled normally

2019-07-16 Thread GitBox
morningman opened a new pull request #1489: Remove query status report from BE when query is cancelled normally URL: https://github.com/apache/incubator-doris/pull/1489 When query result reach limit, the Coordinator in FE will send a cancel request to BE to cancel the query. And when bei

[GitHub] [incubator-doris] EmmyMiao87 opened a new pull request #1488: Add administrator guide of load

2019-07-16 Thread GitBox
EmmyMiao87 opened a new pull request #1488: Add administrator guide of load URL: https://github.com/apache/incubator-doris/pull/1488 The catalogue of load docs: + load-manual.md broker-load-manual.md insert-into-manual.md stream-load-manual.md This commit a

[GitHub] [incubator-doris] morningman merged pull request #1472: Support grant GRANT_PRIV on database or table level

2019-07-16 Thread GitBox
morningman merged pull request #1472: Support grant GRANT_PRIV on database or table level URL: https://github.com/apache/incubator-doris/pull/1472 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-doris] morningman merged pull request #1487: Modify the result json format of mini load

2019-07-16 Thread GitBox
morningman merged pull request #1487: Modify the result json format of mini load URL: https://github.com/apache/incubator-doris/pull/1487 This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [incubator-doris] imay commented on a change in pull request #1472: Support grant GRANT_PRIV on database or table level

2019-07-16 Thread GitBox
imay commented on a change in pull request #1472: Support grant GRANT_PRIV on database or table level URL: https://github.com/apache/incubator-doris/pull/1472#discussion_r303817273 ## File path: docs/help/Contents/Account Management/help.md ## @@ -109,6 +109,7 @@ privileg

[GitHub] [incubator-doris] imay commented on a change in pull request #1472: Support grant GRANT_PRIV on database or table level

2019-07-16 Thread GitBox
imay commented on a change in pull request #1472: Support grant GRANT_PRIV on database or table level URL: https://github.com/apache/incubator-doris/pull/1472#discussion_r303819303 ## File path: fe/src/main/java/org/apache/doris/analysis/GrantStmt.java ## @@ -92,32 +91,51

[GitHub] [incubator-doris] imay commented on a change in pull request #1487: Modify the result json format of mini load

2019-07-16 Thread GitBox
imay commented on a change in pull request #1487: Modify the result json format of mini load URL: https://github.com/apache/incubator-doris/pull/1487#discussion_r303789164 ## File path: be/src/exec/tablet_sink.h ## @@ -168,6 +168,9 @@ class OlapTableSink : public DataSink