You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "levy5307 (via GitHub)" <gi...@apache.org> on 2023/03/20 12:02:26 UTC
[GitHub] [doris] levy5307 opened a new pull request, #17967: [fix](fragment mgr): check duplicated pipeline fragment instance correctly
levy5307 opened a new pull request, #17967:
URL: https://github.com/apache/doris/pull/17967
# Proposed changes
Issue Number: close #xxx
## Problem summary
`_fragment_map` has no bussiness with the pipeline version of `exec_plan_fragment` function. So we can't use it to check whether a duplicated fragment instance is coming.
## Checklist(Required)
* [ ] Does it affect the original behavior
* [ x ] Has unit tests been added
* [ ] Has document been added or modified
* [ ] Does it need to update dependencies
* [ x ] Is this PR support rollback (If NO, please explain WHY)
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] hello-stephen commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1479287377
TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.03 seconds
stream load tsv: 473 seconds loaded 74807831229 Bytes, about 150 MB/s
stream load json: 24 seconds loaded 2358488459 Bytes, about 93 MB/s
stream load orc: 76 seconds loaded 1101869774 Bytes, about 13 MB/s
stream load parquet: 33 seconds loaded 861443392 Bytes, about 24 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230322102219_clickbench_pr_118583.html
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] github-actions[bot] commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1478877461
clang-tidy review says "All clean, LGTM! :+1:"
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1477325911
run buildall
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1482175103
run buildall
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1477811378
run buildall
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1479460207
run buildall
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1478883832
run buildall
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1480938603
run buildall
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] github-actions[bot] commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1518926857
clang-tidy review says "All clean, LGTM! :+1:"
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1477456037
run p0
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1480529926
run buildall
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1480482788
run buildall
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] github-actions[bot] commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1476106591
clang-tidy review says "All clean, LGTM! :+1:"
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] github-actions[bot] commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1482292859
clang-tidy review says "All clean, LGTM! :+1:"
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 closed pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 closed pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
URL: https://github.com/apache/doris/pull/17967
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1477218712
run p0
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1477175429
run buildall
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1478829628
run buildall
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1484357977
run buildall
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1478967414
run buildall
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1482160080
run p0
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1482418005
run buildall
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] levy5307 commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "levy5307 (via GitHub)" <gi...@apache.org>.
levy5307 commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1478977837
run buildall
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] github-actions[bot] commented on pull request #17967: [fix](fragment mgr) check duplicated pipeline fragment instance correctly
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #17967:
URL: https://github.com/apache/doris/pull/17967#issuecomment-1476116034
clang-tidy review says "All clean, LGTM! :+1:"
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org