yangzhg merged pull request #6538:
URL: https://github.com/apache/incubator-doris/pull/6538
--
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.
To unsubscribe, e-mail: commit
This is an automated email from the ASF dual-hosted git repository.
yangzhg pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-doris.git
The following commit(s) were added to refs/heads/master by this push:
new 9f7d4cf [BUG] fix bugs with string ty
yangzhg commented on a change in pull request #6504:
URL: https://github.com/apache/incubator-doris/pull/6504#discussion_r699980646
##
File path: be/src/exprs/json_functions.cpp
##
@@ -108,6 +109,96 @@ DoubleVal JsonFunctions::get_json_double(FunctionContext*
context, const St
yangzhg commented on a change in pull request #6504:
URL: https://github.com/apache/incubator-doris/pull/6504#discussion_r699981146
##
File path: be/src/exprs/json_functions.cpp
##
@@ -108,6 +109,96 @@ DoubleVal JsonFunctions::get_json_double(FunctionContext*
context, const St
qzsee commented on pull request #6526:
URL: https://github.com/apache/incubator-doris/pull/6526#issuecomment-910058088
> This bug has been fixed in new version
Yesterday, I tested the latest master branch and did not have this problem.
However, version 0.14 still has this problem. Ar
qzsee commented on pull request #6526:
URL: https://github.com/apache/incubator-doris/pull/6526#issuecomment-910064230
> ```c++
> Status ExprContext::prepare(RuntimeState* state, const RowDescriptor&
row_desc,
> const std::shared_ptr& tracker) {
> _p
EmmyMiao87 commented on a change in pull request #6539:
URL: https://github.com/apache/incubator-doris/pull/6539#discussion_r68628
##
File path: be/src/runtime/file_result_writer.cpp
##
@@ -39,13 +42,36 @@ namespace doris {
const size_t FileResultWriter::OUTSTREAM_BUFFER
qzsee edited a comment on pull request #6526:
URL: https://github.com/apache/incubator-doris/pull/6526#issuecomment-910064230
> ```c++
> Status ExprContext::prepare(RuntimeState* state, const RowDescriptor&
row_desc,
> const std::shared_ptr& tracker) {
>
qzsee commented on issue #6525:
URL:
https://github.com/apache/incubator-doris/issues/6525#issuecomment-910067507
> 新版本的 runtime filter 取消了这部分逻辑了。所以新版本没问题。
> 旧版本的 runtime filter 确实有可能有你这个问题。
是的，但是新版本的runtime
filter，也有IN类型的过滤，这部分同样有复制谓词的过程，如果_tuple_idx不清空，不太清楚有些特别的查询是否继续命中这种bug。
EmmyMiao87 commented on a change in pull request #6539:
URL: https://github.com/apache/incubator-doris/pull/6539#discussion_r68628
##
File path: be/src/runtime/file_result_writer.cpp
##
@@ -39,13 +42,36 @@ namespace doris {
const size_t FileResultWriter::OUTSTREAM_BUFFER
caiconghui commented on a change in pull request #6416:
URL: https://github.com/apache/incubator-doris/pull/6416#discussion_r699112144
##
File path: fe/fe-core/src/main/java/org/apache/doris/alter/Alter.java
##
@@ -260,17 +251,11 @@ private void processModifyColumnComment(Datab
zbtzbtzbt commented on pull request #6540:
URL: https://github.com/apache/incubator-doris/pull/6540#issuecomment-909908624
you can add some descriptions about this pr.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
zhao447131724 closed issue #6537:
URL: https://github.com/apache/incubator-doris/issues/6537
--
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.
To unsubscribe, e-mail: commi
caiconghui commented on issue #6535:
URL:
https://github.com/apache/incubator-doris/issues/6535#issuecomment-909949619
you can put this question to github discussion
https://github.com/apache/incubator-doris/discussions
--
This is an automated message from the Apache Git Service.
To res
crazy2323 closed issue #6530:
URL: https://github.com/apache/incubator-doris/issues/6530
--
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.
To unsubscribe, e-mail: commits-u
BiteThet commented on pull request #6526:
URL: https://github.com/apache/incubator-doris/pull/6526#issuecomment-909937505
```cpp
Status ExprContext::prepare(RuntimeState* state, const RowDescriptor&
row_desc,
const std::shared_ptr& tracker) {
_pr
github-actions[bot] commented on pull request #6455:
URL: https://github.com/apache/incubator-doris/pull/6455#issuecomment-909818793
--
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 c
yangzhg commented on issue #6498:
URL:
https://github.com/apache/incubator-doris/issues/6498#issuecomment-909805608
I will evaluate this feature later, and you are welcome to contribute to
this feature
--
This is an automated message from the Apache Git Service.
To respond to the messag
EmmyMiao87 commented on pull request #6540:
URL: https://github.com/apache/incubator-doris/pull/6540#issuecomment-909917654
Your pr contains more than 30,000 lines of code, but there is no commit msg,
so this won't work.
And the title is too big.
It is best to submit them separat
morningman commented on a change in pull request #6102:
URL: https://github.com/apache/incubator-doris/pull/6102#discussion_r699283182
##
File path: fe/fe-core/src/main/java/org/apache/doris/catalog/Catalog.java
##
@@ -3096,6 +3104,48 @@ public void createTableLike(CreateTableL
yangzhg commented on a change in pull request #6504:
URL: https://github.com/apache/incubator-doris/pull/6504#discussion_r699800543
##
File path: be/src/exprs/json_functions.cpp
##
@@ -108,6 +108,96 @@ DoubleVal JsonFunctions::get_json_double(FunctionContext*
context, const St
github-actions[bot] commented on pull request #6538:
URL: https://github.com/apache/incubator-doris/pull/6538#issuecomment-909808383
--
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 c
EmmyMiao87 commented on a change in pull request #6540:
URL: https://github.com/apache/incubator-doris/pull/6540#discussion_r699864015
##
File path: manage/www/.babelrc
##
@@ -0,0 +1,25 @@
+{
Review comment:
The code needs to meet development specifications. Such as lic
yangzhg commented on pull request #6526:
URL: https://github.com/apache/incubator-doris/pull/6526#issuecomment-909935262
This bug has been fixed in new version
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
github-actions[bot] commented on pull request #6524:
URL: https://github.com/apache/incubator-doris/pull/6524#issuecomment-909801267
--
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 c
github-actions[bot] commented on pull request #6506:
URL: https://github.com/apache/incubator-doris/pull/6506#issuecomment-909876498
--
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 c
caiconghui commented on issue #6536:
URL:
https://github.com/apache/incubator-doris/issues/6536#issuecomment-909950312
you can tell more details about this issue in github discussion.
https://github.com/apache/incubator-doris/discussions
--
This is an automated message from the Apache
qzsee edited a comment on pull request #6526:
URL: https://github.com/apache/incubator-doris/pull/6526#issuecomment-910064230
> ```c++
> Status ExprContext::prepare(RuntimeState* state, const RowDescriptor&
row_desc,
> const std::shared_ptr& tracker) {
>
EmmyMiao87 commented on issue #6525:
URL:
https://github.com/apache/incubator-doris/issues/6525#issuecomment-909941736
新版本的 runtime filter 取消了这部分逻辑了。所以新版本没问题。
旧版本的 runtime filter 确实有可能有你这个问题。
--
This is an automated message from the Apache Git Service.
To respond to the message, plea
yangzhg edited a comment on issue #6530:
URL:
https://github.com/apache/incubator-doris/issues/6530#issuecomment-909949369
--
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.
ccoffline commented on a change in pull request #6416:
URL: https://github.com/apache/incubator-doris/pull/6416#discussion_r699104541
##
File path:
fe/fe-core/src/main/java/org/apache/doris/alter/SchemaChangeJobV2.java
##
@@ -218,9 +215,9 @@ protected void runPendingJob() thro
zbtzbtzbt edited a comment on pull request #6540:
URL: https://github.com/apache/incubator-doris/pull/6540#issuecomment-909908624
you can add some descriptions about this pr, the title is so ambiguous
--
This is an automated message from the Apache Git Service.
To respond to the message,
yangzhg commented on a change in pull request #6532:
URL: https://github.com/apache/incubator-doris/pull/6532#discussion_r699840263
##
File path:
fe/fe-core/src/main/java/org/apache/doris/http/action/WebBaseAction.java
##
@@ -67,12 +67,12 @@
+ " rel=\"stylesheet\
morningman commented on a change in pull request #6539:
URL: https://github.com/apache/incubator-doris/pull/6539#discussion_r699833595
##
File path: fe/fe-core/src/main/java/org/apache/doris/analysis/OutFileClause.java
##
@@ -45,6 +48,8 @@
import java.util.Set;
import java.ut
github-actions[bot] commented on pull request #6534:
URL: https://github.com/apache/incubator-doris/pull/6534#issuecomment-909124987
--
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 c
yangzhg merged pull request #6534:
URL: https://github.com/apache/incubator-doris/pull/6534
--
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.
To unsubscribe, e-mail: commit
zhao447131724 removed a comment on issue #6537:
URL:
https://github.com/apache/incubator-doris/issues/6537#issuecomment-908985808
直接使用mysql客户端连接是可以的，但是使用flink sql会报这个错误
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
morningman merged pull request #6475:
URL: https://github.com/apache/incubator-doris/pull/6475
--
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.
To unsubscribe, e-mail: com
yangzhg commented on issue #6535:
URL:
https://github.com/apache/incubator-doris/issues/6535#issuecomment-909948112
36字节是个经验值，varchar 截断是因为不好判断varchar 的长度
--
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
yangzhg merged pull request #6538:
URL: https://github.com/apache/incubator-doris/pull/6538
--
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.
To unsubscribe, e-mail: commit
yangzhg commented on issue #6530:
URL:
https://github.com/apache/incubator-doris/issues/6530#issuecomment-909949369
you shoud post all more build log
--
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 g
qzsee commented on issue #6525:
URL:
https://github.com/apache/incubator-doris/issues/6525#issuecomment-910067507
> 新版本的 runtime filter 取消了这部分逻辑了。所以新版本没问题。
> 旧版本的 runtime filter 确实有可能有你这个问题。
是的，但是新版本的runtime
filter，也有IN类型的过滤，这部分同样有复制谓词的过程，如果_tuple_idx不清空，不太清楚有些特别的查询是否继续命中这种bug。
qzsee commented on pull request #6526:
URL: https://github.com/apache/incubator-doris/pull/6526#issuecomment-910058088
--
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.
To un
caiconghui merged pull request #6524:
URL: https://github.com/apache/incubator-doris/pull/6524
--
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.
To unsubscribe, e-mail: com
github-actions[bot] commented on pull request #6480:
URL: https://github.com/apache/incubator-doris/pull/6480#issuecomment-909083588
--
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 c
crazy2323 commented on issue #6530:
URL:
https://github.com/apache/incubator-doris/issues/6530#issuecomment-909950031
upgrade gcc version
--
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-actions[bot] commented on pull request #6475:
URL: https://github.com/apache/incubator-doris/pull/6475#issuecomment-909189880
--
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 c
morningman commented on a change in pull request #6416:
URL: https://github.com/apache/incubator-doris/pull/6416#discussion_r699260553
##
File path: fe/fe-core/src/main/java/org/apache/doris/alter/RollupJobV2.java
##
@@ -667,22 +638,26 @@ private void replayCancelled(RollupJobV
ccoffline closed issue #6432:
URL: https://github.com/apache/incubator-doris/issues/6432
--
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.
To unsubscribe, e-mail: commits-u
ccoffline commented on issue #6432:
URL:
https://github.com/apache/incubator-doris/issues/6432#issuecomment-910146278
fixed by #4081
--
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
yangzhg commented on pull request #6526:
URL: https://github.com/apache/incubator-doris/pull/6526#issuecomment-910165007
> > This bug has been fixed in new version
>
> Yesterday, I tested the latest master branch and did not have this
problem. However, version 0.14 still has this pro
dh-cloud opened a new pull request #6544:
URL: https://github.com/apache/incubator-doris/pull/6544
cluster info: 4 hosts (10.35.15.164,165,206,207), 8 bes , table with
colocate_with, 3 replicas
version: 0.14
we found colocate join table replica can't repaired. the tablet sta
github-actions[bot] commented on pull request #6504:
URL: https://github.com/apache/incubator-doris/pull/6504#issuecomment-910212271
--
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 c
zbtzbtzbt commented on pull request #6518:
URL: https://github.com/apache/incubator-doris/pull/6518#issuecomment-910255419
here is the final version.
--
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
morningman commented on a change in pull request #6416:
URL: https://github.com/apache/incubator-doris/pull/6416#discussion_r700266994
##
File path: fe/fe-core/src/main/java/org/apache/doris/analysis/LoadStmt.java
##
@@ -330,17 +329,12 @@ public void analyze(Analyzer analyzer)
Skysheepwang opened a new issue #6545:
URL: https://github.com/apache/incubator-doris/issues/6545
### 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
morningman commented on pull request #6544:
URL: https://github.com/apache/incubator-doris/pull/6544#issuecomment-910401124
Why you deploy 2 BEs on one host?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
BiteThet commented on pull request #6540:
URL: https://github.com/apache/incubator-doris/pull/6540#issuecomment-910406641
--
This is an automated message from the Apache Gi
morningman commented on a change in pull request #6539:
URL: https://github.com/apache/incubator-doris/pull/6539#discussion_r700356119
##
File path: be/src/runtime/file_result_writer.cpp
##
@@ -39,13 +42,36 @@ namespace doris {
const size_t FileResultWriter::OUTSTREAM_BUFFER
github-actions[bot] commented on pull request #5781:
URL: https://github.com/apache/incubator-doris/pull/5781#issuecomment-910440916
--
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 c
dh-cloud commented on pull request #6544:
URL: https://github.com/apache/incubator-doris/pull/6544#issuecomment-910978995
> Why you deploy 2 BEs on one host?
put be on one ssd，we test 2 num be on one host, the performance is the
better.
may be test case is not good，we should test agai
dh-cloud removed a comment on pull request #6544:
URL: https://github.com/apache/incubator-doris/pull/6544#issuecomment-910978995
> Why you deploy 2 BEs on one host?
put be on one ssd，we test 2 num be on one host, the performance is the
better.
may be test case is not good，we should t
dh-cloud commented on pull request #6544:
URL: https://github.com/apache/incubator-doris/pull/6544#issuecomment-910983438
> Why you deploy 2 BEs on one host?
we put one be on one disk，test 2 bes on one host, the performance is better
may be the test case is not good。 next we should
dh-cloud edited a comment on pull request #6544:
URL: https://github.com/apache/incubator-doris/pull/6544#issuecomment-910983438
> Why you deploy 2 BEs on one host?
we put one be on one disk，test 2 bes on one host, the performance is better
may be the test case is not good。 next we
xiaoxiaopan118 commented on a change in pull request #6532:
URL: https://github.com/apache/incubator-doris/pull/6532#discussion_r700674900
##
File path:
fe/fe-core/src/main/java/org/apache/doris/http/action/WebBaseAction.java
##
@@ -67,12 +67,12 @@
+ " rel=\"styl
yangzhg merged pull request #6506:
URL: https://github.com/apache/incubator-doris/pull/6506
--
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.
To unsubscribe, e-mail: commit
This is an automated email from the ASF dual-hosted git repository.
yangzhg pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-doris.git.
from 9f7d4cf [BUG] fix bugs with string type (#6538)
add 4dd610c [Feature] Support for storage layer benc
This is an automated email from the ASF dual-hosted git repository.
yangzhg pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-doris.git
The following commit(s) were added to refs/heads/master by this push:
new 7a15e58 [Feature]Support functions o
yangzhg merged pull request #6504:
URL: https://github.com/apache/incubator-doris/pull/6504
--
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.
To unsubscribe, e-mail: commit
yangzhg merged pull request #6455:
URL: https://github.com/apache/incubator-doris/pull/6455
--
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.
To unsubscribe, e-mail: commit
This is an automated email from the ASF dual-hosted git repository.
yangzhg pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-doris.git
The following commit(s) were added to refs/heads/master by this push:
new d8cde8c (#6454) Remove useless code f
yangzhg merged pull request #5781:
URL: https://github.com/apache/incubator-doris/pull/5781
--
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.
To unsubscribe, e-mail: commit
yangzhg closed issue #5779:
URL: https://github.com/apache/incubator-doris/issues/5779
--
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.
To unsubscribe, e-mail: commits-uns
This is an automated email from the ASF dual-hosted git repository.
yangzhg pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-doris.git
The following commit(s) were added to refs/heads/master by this push:
new 5719995 [Compaction][ThreadPool]Suppo
wangliansong opened a new pull request #6546:
URL: https://github.com/apache/incubator-doris/pull/6546
stripe_outer_array to strip_outer_array
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull req
github-actions[bot] commented on pull request #6546:
URL: https://github.com/apache/incubator-doris/pull/6546#issuecomment-911090797
--
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 c
srikker opened a new pull request #6547:
URL: https://github.com/apache/incubator-doris/pull/6547
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature reque
EmmyMiao87 commented on a change in pull request #6539:
URL: https://github.com/apache/incubator-doris/pull/6539#discussion_r700721088
##
File path: be/src/runtime/file_result_writer.cpp
##
@@ -39,13 +42,36 @@ namespace doris {
const size_t FileResultWriter::OUTSTREAM_BUFFER
wangbo merged pull request #6480:
URL: https://github.com/apache/incubator-doris/pull/6480
--
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.
To unsubscribe, e-mail: commits
This is an automated email from the ASF dual-hosted git repository.
wangbo pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-doris.git.
from 5719995 [Compaction][ThreadPool]Support adjust compaction threads num
at runtime (#5781)
add 6ac0ab6
wangshuo128 commented on a change in pull request #6518:
URL: https://github.com/apache/incubator-doris/pull/6518#discussion_r696724744
##
File path:
fe/fe-core/src/main/java/org/apache/doris/rewrite/CompoundPredicateWriteRule.java
##
@@ -0,0 +1,134 @@
+// Licensed to the Apac
wangshuo128 commented on a change in pull request #6518:
URL: https://github.com/apache/incubator-doris/pull/6518#discussion_r700742327
##
File path: fe/pom.xml
##
@@ -661,7 +661,7 @@ under the License.
org.projectlombok
lombok
-
wangshuo128 commented on a change in pull request #6518:
URL: https://github.com/apache/incubator-doris/pull/6518#discussion_r700760493
##
File path:
fe/fe-core/src/main/java/org/apache/doris/rewrite/CompoundPredicateWriteRule.java
##
@@ -0,0 +1,102 @@
+// Licensed to the Apac
83 matches
Mail list logo
