[GitHub] [incubator-doris] yangzhg commented on a change in pull request #5547: [Compile][Refactor] fix some warning in gcc higher than 7

2021-03-21 Thread GitBox
yangzhg commented on a change in pull request #5547: URL: https://github.com/apache/incubator-doris/pull/5547#discussion_r598446741 ## File path: be/src/olap/decimal12.h ## @@ -156,6 +139,8 @@ struct decimal12_t { int32_t fraction; } __attribute__((packed)); +static_ass

[GitHub] [incubator-doris] xinghuayu007 closed pull request #5436: [FEATURE]Check date type to avoid scan all partitions

2021-03-21 Thread GitBox
xinghuayu007 closed pull request #5436: URL: https://github.com/apache/incubator-doris/pull/5436 -- 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. For queries about this se

[GitHub] [incubator-doris] yangzhg commented on a change in pull request #5547: [Compile][Refactor] fix some warning in gcc higher than 7

2021-03-21 Thread GitBox
yangzhg commented on a change in pull request #5547: URL: https://github.com/apache/incubator-doris/pull/5547#discussion_r598446741 ## File path: be/src/olap/decimal12.h ## @@ -156,6 +139,8 @@ struct decimal12_t { int32_t fraction; } __attribute__((packed)); +static_ass

[GitHub] [incubator-doris] stdpain opened a new pull request #5547: [Compile][Refactor ] fix some warning in gcc higher than 7

2021-03-21 Thread GitBox
stdpain opened a new pull request #5547: URL: https://github.com/apache/incubator-doris/pull/5547 make decimal12_t as a pod type ## Proposed changes Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fi

[GitHub] [incubator-doris] stdpain opened a new issue #5546: [Compile] Some compile warning in gcc > 8

2021-03-21 Thread GitBox
stdpain opened a new issue #5546: URL: https://github.com/apache/incubator-doris/issues/5546 **Describe the bug** clearing an object of type 'struct ' with no trivial copy-assignment; use assignment or value-initialization instead **Expected behavior** no warning here

[GitHub] [incubator-doris] xinghuayu007 commented on a change in pull request #5436: [FEATURE]Check date type to avoid scan all partitions

2021-03-21 Thread GitBox
xinghuayu007 commented on a change in pull request #5436: URL: https://github.com/apache/incubator-doris/pull/5436#discussion_r598409824 ## File path: fe/fe-core/src/main/java/org/apache/doris/rewrite/SimplifyInvalidDateBinaryPredicatesDateRule.java ## @@ -0,0 +1,54 @@ +// Lic

[GitHub] [incubator-doris] EmmyMiao87 commented on pull request #5521: [Colocate plan][Step1] Colocate join covers more situations

2021-03-21 Thread GitBox
EmmyMiao87 commented on pull request #5521: URL: https://github.com/apache/incubator-doris/pull/5521#issuecomment-803734159 > the comment is same as line 570. just keep one is enough. removed -- This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [incubator-doris] EmmyMiao87 commented on pull request #5521: [Colocate plan][Step1] Colocate join covers more situations

2021-03-21 Thread GitBox
EmmyMiao87 commented on pull request #5521: URL: https://github.com/apache/incubator-doris/pull/5521#issuecomment-803734110 > seems not need return null too many, may under code is better: > > ``` >if (this instanceof ScanNode && tupleIds.contains(tupleId)) { >

[GitHub] [incubator-doris] EmmyMiao87 commented on pull request #5521: [Colocate plan][Step1] Colocate join covers more situations

2021-03-21 Thread GitBox
EmmyMiao87 commented on pull request #5521: URL: https://github.com/apache/incubator-doris/pull/5521#issuecomment-803728778 > Why change import order of LogManager? The order of import should comply with http://doris.apache.org/master/zh-CN/developer-guide/fe-eclipse-dev.html#%E4%BB%

[GitHub] [incubator-doris] EmmyMiao87 commented on a change in pull request #5521: [Colocate plan][Step1] Colocate join covers more situations

2021-03-21 Thread GitBox
EmmyMiao87 commented on a change in pull request #5521: URL: https://github.com/apache/incubator-doris/pull/5521#discussion_r598395644 ## File path: fe/fe-core/src/main/java/org/apache/doris/system/SystemInfoService.java ## @@ -44,6 +41,10 @@ import com.google.common.collect.

[incubator-doris] branch master updated (4085b19 -> 253056c)

2021-03-21 Thread lingmiao
This is an automated email from the ASF dual-hosted git repository. lingmiao pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-doris.git. from 4085b19 [Bug] Fix multi broker load bug (#5543) add 253056c [Docs] Modify schema change sql referen

[GitHub] [incubator-doris] EmmyMiao87 merged pull request #5530: [Docs] Modify schema change sql reference

2021-03-21 Thread GitBox
EmmyMiao87 merged pull request #5530: URL: https://github.com/apache/incubator-doris/pull/5530 -- 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. For queries about this serv

[GitHub] [incubator-doris] morningman closed issue #5544: [Bug] Failed to execute multi load

2021-03-21 Thread GitBox
morningman closed issue #5544: URL: https://github.com/apache/incubator-doris/issues/5544 -- 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. For queries about this service,

[incubator-doris] branch master updated (c97ddd7 -> 4085b19)

2021-03-21 Thread morningman
This is an automated email from the ASF dual-hosted git repository. morningman pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-doris.git. from c97ddd7 [Enhance] Show brokers' hostname (#5522) add 4085b19 [Bug] Fix multi broker load bug (#55

[GitHub] [incubator-doris] morningman merged pull request #5543: [Bug] Fix multi broker load bug

2021-03-21 Thread GitBox
morningman merged pull request #5543: URL: https://github.com/apache/incubator-doris/pull/5543 -- 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. For queries about this serv

[GitHub] [incubator-doris] morningman commented on pull request #5452: [Audit][Stream Load] Support audit function for stream load

2021-03-21 Thread GitBox
morningman commented on pull request #5452: URL: https://github.com/apache/incubator-doris/pull/5452#issuecomment-803596560 And I pulled and run your PR, submit several stream load. I can see log `put stream_load_record rocksdb successfully. label: f0130520-835a-4d05-a38d-95f67f5b8c7b` i

[GitHub] [incubator-doris] morningman commented on a change in pull request #5452: [Audit][Stream Load] Support audit function for stream load

2021-03-21 Thread GitBox
morningman commented on a change in pull request #5452: URL: https://github.com/apache/incubator-doris/pull/5452#discussion_r598289847 ## File path: fe/fe-core/src/main/java/org/apache/doris/load/StreamLoadRecordMgr.java ## @@ -0,0 +1,112 @@ +// Licensed to the Apache Software

[GitHub] [incubator-doris] morningman commented on a change in pull request #5452: [Audit][Stream Load] Support audit function for stream load

2021-03-21 Thread GitBox
morningman commented on a change in pull request #5452: URL: https://github.com/apache/incubator-doris/pull/5452#discussion_r598279486 ## File path: be/src/common/config.h ## @@ -348,6 +348,12 @@ CONF_mInt32(streaming_load_rpc_max_alive_time_sec, "1200"); CONF_Int32(tablet_wri

[incubator-doris] branch master updated (a91888a -> c97ddd7)

2021-03-21 Thread morningman
This is an automated email from the ASF dual-hosted git repository. morningman pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-doris.git. from a91888a [BUG] fix memory limit failure and optimize memory usage in join stage (#5514) add c97ddd

[GitHub] [incubator-doris] morningman merged pull request #5522: [Enhance] Show brokers' hostname

2021-03-21 Thread GitBox
morningman merged pull request #5522: URL: https://github.com/apache/incubator-doris/pull/5522 -- 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. For queries about this serv

svn commit: r46699 - in /dev/incubator/doris/0.14/0.14.0-rc03: ./ apache-doris-0.14.0-incubating-src.tar.gz apache-doris-0.14.0-incubating-src.tar.gz.asc apache-doris-0.14.0-incubating-src.tar.gz.sha5

2021-03-21 Thread yangzhg
Author: yangzhg Date: Sun Mar 21 10:30:42 2021 New Revision: 46699 Log: Add 0.14.0-rc03 Added: dev/incubator/doris/0.14/0.14.0-rc03/ dev/incubator/doris/0.14/0.14.0-rc03/apache-doris-0.14.0-incubating-src.tar.gz (with props) dev/incubator/doris/0.14/0.14.0-rc03/apache-doris-0.14.

[incubator-doris] annotated tag 0.14.0-rc03 updated (2f50015 -> f282301)

2021-03-21 Thread yangzhg
This is an automated email from the ASF dual-hosted git repository. yangzhg pushed a change to annotated tag 0.14.0-rc03 in repository https://gitbox.apache.org/repos/asf/incubator-doris.git. *** WARNING: tag 0.14.0-rc03 was modified! *** from 2f50015 (commit) to f282301 (tag) tagg

[incubator-doris] annotated tag 0.14.0-rc03 updated (2f50015 -> 5cbf00e)

2021-03-21 Thread yangzhg
This is an automated email from the ASF dual-hosted git repository. yangzhg pushed a change to annotated tag 0.14.0-rc03 in repository https://gitbox.apache.org/repos/asf/incubator-doris.git. *** WARNING: tag 0.14.0-rc03 was modified! *** from 2f50015 (commit) to 5cbf00e (tag) tagg

[incubator-doris] 02/03: [Apache] Add DISCLAIMER back (#5531)

2021-03-21 Thread yangzhg
This is an automated email from the ASF dual-hosted git repository. yangzhg pushed a commit to branch branch-0.14 in repository https://gitbox.apache.org/repos/asf/incubator-doris.git commit 3de1d507db6eed41263e09767902689147976bde Author: Mingyu Chen AuthorDate: Thu Mar 18 11:24:19 2021 +0800

[incubator-doris] 03/03: [Bug] Fix bug that the stale rowset file will not be deleted (#5527)

2021-03-21 Thread yangzhg
This is an automated email from the ASF dual-hosted git repository. yangzhg pushed a commit to branch branch-0.14 in repository https://gitbox.apache.org/repos/asf/incubator-doris.git commit a2800c8a22c1054eaafdeacf86abb26d0cef2e9e Author: Mingyu Chen AuthorDate: Wed Mar 17 22:31:05 2021 +0800

[incubator-doris] 01/03: [Bug] Fix the memory expand 10~1000x of compression algorithm (#5504)

2021-03-21 Thread yangzhg
This is an automated email from the ASF dual-hosted git repository. yangzhg pushed a commit to branch branch-0.14 in repository https://gitbox.apache.org/repos/asf/incubator-doris.git commit f1860f7258338a665750fb27c3079146ed3a45db Author: HappenLee AuthorDate: Fri Mar 12 23:04:07 2021 +0800

[incubator-doris] branch branch-0.14 updated (9aba744 -> a2800c8)

2021-03-21 Thread yangzhg
This is an automated email from the ASF dual-hosted git repository. yangzhg pushed a change to branch branch-0.14 in repository https://gitbox.apache.org/repos/asf/incubator-doris.git. from 9aba744 [Bug] Fix bug that data lost when doing backup job (#5473) new f1860f7 [Bug] Fix the m

[GitHub] [incubator-doris] yangzhg opened a new pull request #5545: remove unused char at end of file

2021-03-21 Thread GitBox
yangzhg opened a new pull request #5545: URL: https://github.com/apache/incubator-doris/pull/5545 ## Proposed changes remove unused char at end of file ## Types of changes What types of changes does your code introduce to Doris? _Put an `x` in the boxes that apply_

[GitHub] [incubator-doris] yangzhg commented on a change in pull request #5543: [Bug] Fix multi broker load bug

2021-03-21 Thread GitBox
yangzhg commented on a change in pull request #5543: URL: https://github.com/apache/incubator-doris/pull/5543#discussion_r598242095 ## File path: fe/fe-core/src/main/java/org/apache/doris/analysis/BrokerDesc.java ## @@ -60,7 +60,11 @@ public BrokerDesc(String name, Map propert

[GitHub] [incubator-doris] yangzhg commented on a change in pull request #5543: [Bug] Fix multi broker load bug

2021-03-21 Thread GitBox
yangzhg commented on a change in pull request #5543: URL: https://github.com/apache/incubator-doris/pull/5543#discussion_r598242095 ## File path: fe/fe-core/src/main/java/org/apache/doris/analysis/BrokerDesc.java ## @@ -60,7 +60,11 @@ public BrokerDesc(String name, Map propert