[GitHub] [doris] hello-stephen commented on pull request #14422: [test](jdbc table)add new mysql jdbc external table

2022-11-19 Thread GitBox
hello-stephen commented on PR #14422: URL: https://github.com/apache/doris/pull/14422#issuecomment-1321062821 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 35.76 seconds load time: 435 seconds storage size: 17151083938 Bytes https://doris-co

[GitHub] [doris] hello-stephen commented on pull request #14421: [opt](nereids) make fragment id in explain get inline with profile

2022-11-19 Thread GitBox
hello-stephen commented on PR #14421: URL: https://github.com/apache/doris/pull/14421#issuecomment-1321062065 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 34.5 seconds load time: 436 seconds storage size: 17150514423 Bytes https://doris-com

[GitHub] [doris] zhannngchen commented on pull request #14418: [fix](exec) fix thread token shutdown

2022-11-19 Thread GitBox
zhannngchen commented on PR #14418: URL: https://github.com/apache/doris/pull/14418#issuecomment-1321060430 > Just modify some logs? removed `_set_scan_concurrency(params, fragments_ctx.get())`, this method call will set _thread_token duplicately. -- This is an automated message fr

[GitHub] [doris] lsy3993 opened a new pull request, #14422: [test](jdbc table)add new mysql jdbc external table

2022-11-19 Thread GitBox
lsy3993 opened a new pull request, #14422: URL: https://github.com/apache/doris/pull/14422 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original behavior: - [ ] Yes

[GitHub] [doris] englefly opened a new pull request, #14421: [opt](nereids) make fragment id in explain get inline with profile

2022-11-19 Thread GitBox
englefly opened a new pull request, #14421: URL: https://github.com/apache/doris/pull/14421 # Proposed changes Nereids assign fragment ID in its own way. The fragment Id in explain is different from the fragment id in profile. This difference makes trouble to understand profile.

[GitHub] [doris] xinyiZzz opened a new pull request, #14419: [Docs](memory) Admin-manual adds mem tracker, memory exceeds limit, OOM analysis

2022-11-19 Thread GitBox
xinyiZzz opened a new pull request, #14419: URL: https://github.com/apache/doris/pull/14419 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original behavior: - [ ] Ye

[GitHub] [doris] yinzhijian commented on a diff in pull request #14377: [improvement](planner) eliminating useless sort node

2022-11-19 Thread GitBox
yinzhijian commented on code in PR #14377: URL: https://github.com/apache/doris/pull/14377#discussion_r1027225986 ## fe/fe-core/src/main/java/org/apache/doris/qe/SessionVariable.java: ## @@ -586,6 +588,9 @@ public class SessionVariable implements Serializable, Writable { @

[doris] branch master updated: [typo](docs)Performance test documentation update (#14147)

2022-11-19 Thread jiafengzheng
This is an automated email from the ASF dual-hosted git repository. jiafengzheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/doris.git The following commit(s) were added to refs/heads/master by this push: new 71e80e8957 [typo](docs)Performance test doc

[GitHub] [doris] hf200012 merged pull request #14147: [typo](docs)Performance test documentation update

2022-11-19 Thread GitBox
hf200012 merged PR #14147: URL: https://github.com/apache/doris/pull/14147 -- 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-unsubscr...@doris.apa

[GitHub] [doris] github-actions[bot] commented on pull request #8789: [fix] update RewriteBinaryPredicatesRule and consider special decimalLiteral (0.0 is equal to 0)

2022-11-19 Thread GitBox
github-actions[bot] commented on PR #8789: URL: https://github.com/apache/doris/pull/8789#issuecomment-1320997188 We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.

[doris] branch master updated (3489f4826c -> 2ccb5209a0)

2022-11-19 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/doris.git from 3489f4826c [fix](test) sync conf used in pipeline and in repository (#14414) add 2ccb5209a0 (improvement)[doc

[GitHub] [doris] morningman merged pull request #14406: (improvement)[doc] add document version tag instruction

2022-11-19 Thread GitBox
morningman merged PR #14406: URL: https://github.com/apache/doris/pull/14406 -- 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-unsubscr...@doris.a

[doris] branch master updated: [fix](exec) fix thread token shutdown (#14418)

2022-11-19 Thread morningman
This is an automated email from the ASF dual-hosted git repository. morningman pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/doris.git The following commit(s) were added to refs/heads/master by this push: new 3e1e8db173 [fix](exec) fix thread token shutd

[doris] branch master updated (3e1e8db173 -> 3489f4826c)

2022-11-19 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/doris.git from 3e1e8db173 [fix](exec) fix thread token shutdown (#14418) add 3489f4826c [fix](test) sync conf used in pipelin

[GitHub] [doris] morningman merged pull request #14414: [fix](test) sync conf used in pipeline and in repository

2022-11-19 Thread GitBox
morningman merged PR #14414: URL: https://github.com/apache/doris/pull/14414 -- 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-unsubscr...@doris.a

[GitHub] [doris] morningman merged pull request #14418: [fix](exec) fix thread token shutdown

2022-11-19 Thread GitBox
morningman merged PR #14418: URL: https://github.com/apache/doris/pull/14418 -- 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-unsubscr...@doris.a

[GitHub] [doris] HappenLee commented on pull request #14393: [Vectorized](udaf) fix java-udaf couldn't get jar core dump

2022-11-19 Thread GitBox
HappenLee commented on PR #14393: URL: https://github.com/apache/doris/pull/14393#issuecomment-1320914956 Should do the check and return error Status in create function be called ! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[GitHub] [doris] HappenLee commented on pull request #14417: [bug](test) fix regression test of jdbc postgresql table core

2022-11-19 Thread GitBox
HappenLee commented on PR #14417: URL: https://github.com/apache/doris/pull/14417#issuecomment-1320911152 why the fix just fix the case ? -- 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 speci

[GitHub] [doris] HappenLee commented on a diff in pull request #14402: [Optimize](predicate) update inplace on VcompoundPred

2022-11-19 Thread GitBox
HappenLee commented on code in PR #14402: URL: https://github.com/apache/doris/pull/14402#discussion_r1027106252 ## be/src/vec/exprs/vcompound_pred.h: ## @@ -16,38 +16,174 @@ // under the License. #pragma once -#include "runtime/runtime_state.h" +#include + +#include "commo

[GitHub] [doris] HappenLee commented on a diff in pull request #14402: [Optimize](predicate) update inplace on VcompoundPred

2022-11-19 Thread GitBox
HappenLee commented on code in PR #14402: URL: https://github.com/apache/doris/pull/14402#discussion_r1027105960 ## regression-test/conf/regression-conf.groovy: ## @@ -23,11 +23,11 @@ defaultDb = "regression_test" // add useLocalSessionState so that the jdbc will not send // i

[GitHub] [doris] adonis0147 commented on pull request #14410: [chore](macOS) Fix linker errors

2022-11-19 Thread GitBox
adonis0147 commented on PR #14410: URL: https://github.com/apache/doris/pull/14410#issuecomment-1320907944 > > > @adonis0147 In `mem_info.cpp:193`, when it is __APPLE_, there is an empty `refresh_proc_meminfo`, does this not work? > > > > > > Hi @xinyiZzz , the error was caused by

[GitHub] [doris] xinyiZzz commented on pull request #14410: [chore](macOS) Fix linker errors

2022-11-19 Thread GitBox
xinyiZzz commented on PR #14410: URL: https://github.com/apache/doris/pull/14410#issuecomment-1320904440 > > @adonis0147 In `mem_info.cpp:193`, when it is __APPLE_, there is an empty `refresh_proc_meminfo`, does this not work? > > Hi @xinyiZzz , the error was caused by [be/src/servic

[GitHub] [doris] adonis0147 commented on pull request #14410: [chore](macOS) Fix linker errors

2022-11-19 Thread GitBox
adonis0147 commented on PR #14410: URL: https://github.com/apache/doris/pull/14410#issuecomment-1320900276 > @adonis0147 In `mem_info.cpp:193`, when it is __APPLE_, there is an empty `refresh_proc_meminfo`, does this not work? Hi @xinyiZzz , the error was caused by [be/src/service/do

[GitHub] [doris] github-actions[bot] commented on pull request #14147: [typo](docs)Performance test documentation update

2022-11-19 Thread GitBox
github-actions[bot] commented on PR #14147: URL: https://github.com/apache/doris/pull/14147#issuecomment-1320884886 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[doris-website] branch master updated: rename image name

2022-11-19 Thread jiafengzheng
This is an automated email from the ASF dual-hosted git repository. jiafengzheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/doris-website.git The following commit(s) were added to refs/heads/master by this push: new 4decadcd1ef rename image name 4deca

[GitHub] [doris] github-actions[bot] commented on pull request #14418: [fix](exec) fix thread token shutdown

2022-11-19 Thread GitBox
github-actions[bot] commented on PR #14418: URL: https://github.com/apache/doris/pull/14418#issuecomment-1320882883 PR approved by anyone and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [doris] github-actions[bot] commented on pull request #14418: [fix](exec) fix thread token shutdown

2022-11-19 Thread GitBox
github-actions[bot] commented on PR #14418: URL: https://github.com/apache/doris/pull/14418#issuecomment-1320882868 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [doris] xinyiZzz commented on pull request #14410: [chore](macOS) Fix linker errors

2022-11-19 Thread GitBox
xinyiZzz commented on PR #14410: URL: https://github.com/apache/doris/pull/14410#issuecomment-1320879648 @adonis0147 In `mem_info.cpp:193`, when it is __APPLE_, there is an empty `refresh_proc_meminfo`, does this not work? -- This is an automated message from the Apache Git Service. To r

[doris-website] branch master updated: fix benchmark doc

2022-11-19 Thread jiafengzheng
This is an automated email from the ASF dual-hosted git repository. jiafengzheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/doris-website.git The following commit(s) were added to refs/heads/master by this push: new 044672300ec fix benchmark doc 04467

[GitHub] [doris] github-actions[bot] commented on pull request #14147: [typo](docs)Performance test documentation update[dont merge]

2022-11-19 Thread GitBox
github-actions[bot] commented on PR #14147: URL: https://github.com/apache/doris/pull/14147#issuecomment-1320867931 PR approved by anyone and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [doris] github-actions[bot] commented on pull request #14147: [typo](docs)Performance test documentation update[dont merge]

2022-11-19 Thread GitBox
github-actions[bot] commented on PR #14147: URL: https://github.com/apache/doris/pull/14147#issuecomment-1320867924 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[doris-website] 01/02: Performance test documentation update

2022-11-19 Thread jiafengzheng
This is an automated email from the ASF dual-hosted git repository. jiafengzheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/doris-website.git commit 692b07d947c33767e0b1ac2a60e44dd8423a055d Author: jiafeng.zhang AuthorDate: Thu Nov 10 11:42:10 2022 +0800

[doris-website] 02/02: fix

2022-11-19 Thread jiafengzheng
This is an automated email from the ASF dual-hosted git repository. jiafengzheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/doris-website.git commit f1cb9c65e6fe601a9fe56eb7cf767806a145d6fd Author: jiafeng.zhang AuthorDate: Fri Nov 11 16:25:00 2022 +0800

[doris-website] branch master updated (3f4740e1746 -> f1cb9c65e6f)

2022-11-19 Thread jiafengzheng
This is an automated email from the ASF dual-hosted git repository. jiafengzheng pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris-website.git from 3f4740e1746 Datax doris writer doc fix new 692b07d947c Performance test documentation update ne

[GitHub] [doris] dataroaring commented on pull request #14418: [fix](exec) fix thread token shutdown

2022-11-19 Thread GitBox
dataroaring commented on PR #14418: URL: https://github.com/apache/doris/pull/14418#issuecomment-1320866552 Just modify some logs? -- 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 com

[GitHub] [doris] hello-stephen commented on pull request #14418: [fix](exec) fix thread token shutdown

2022-11-19 Thread GitBox
hello-stephen commented on PR #14418: URL: https://github.com/apache/doris/pull/14418#issuecomment-1320845830 TeamCity pipeline, clickbench performance test result: the sum of best hot time: 36.62 seconds load time: 475 seconds storage size: 17170556941 Bytes https://doris-co

[doris] branch master updated (2c42f0a905 -> 5dfe5ef965)

2022-11-19 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/doris.git from 2c42f0a905 [refactor](decimalv3) Refine code for DecimalV3 (#14394) add 5dfe5ef965 [test](hive catalog)add hiv

[GitHub] [doris] morningman merged pull request #14217: [test](hive catalog)add hive catalog test case

2022-11-19 Thread GitBox
morningman merged PR #14217: URL: https://github.com/apache/doris/pull/14217 -- 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-unsubscr...@doris.a

[GitHub] [doris] zhannngchen opened a new pull request, #14418: [fix](exec) fix thread token shutdown

2022-11-19 Thread GitBox
zhannngchen opened a new pull request, #14418: URL: https://github.com/apache/doris/pull/14418 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) 1. Does it affect the original behavior: - [ ]

[GitHub] [doris] Gabriel39 merged pull request #14394: [refactor](decimalv3) Refine code for DecimalV3

2022-11-19 Thread GitBox
Gabriel39 merged PR #14394: URL: https://github.com/apache/doris/pull/14394 -- 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-unsubscr...@doris.ap

[GitHub] [doris] github-actions[bot] commented on pull request #14394: [refactor](decimalv3) Refine code for DecimalV3

2022-11-19 Thread GitBox
github-actions[bot] commented on PR #14394: URL: https://github.com/apache/doris/pull/14394#issuecomment-1320835583 PR approved by anyone and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [doris] github-actions[bot] commented on pull request #14394: [refactor](decimalv3) Refine code for DecimalV3

2022-11-19 Thread GitBox
github-actions[bot] commented on PR #14394: URL: https://github.com/apache/doris/pull/14394#issuecomment-1320835577 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu