You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by GitBox <gi...@apache.org> on 2020/02/21 13:50:38 UTC
[GitHub] [incubator-doris] yangzhg opened a new pull request #2970: fix unit
test randomly failed
yangzhg opened a new pull request #2970: fix unit test randomly failed
URL: https://github.com/apache/incubator-doris/pull/2970
fix http server related unit test failed due to http port has been used
fix unit test failed in build type debug
----------------------------------------------------------------
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, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [incubator-doris] trueeyu commented on a change in pull request
#2970: fix unit test randomly failed
Posted by GitBox <gi...@apache.org>.
trueeyu commented on a change in pull request #2970: fix unit test randomly failed
URL: https://github.com/apache/incubator-doris/pull/2970#discussion_r382891647
##########
File path: be/test/util/arrow/arrow_work_flow_test.cpp
##########
@@ -74,28 +74,30 @@ class ArrowWorkFlowTest : public testing::Test {
TDescriptorTable _t_desc_table;
DescriptorTbl* _desc_tbl;
TPlanNode _tnode;
- ExecEnv _exec_env;
+ ExecEnv* _exec_env;
RuntimeState* _state;
MemTracker *_mem_tracker;
RowDescriptor* _row_desc;
}; // end class ArrowWorkFlowTest
void ArrowWorkFlowTest::init() {
+ _exec_env = ExecEnv::GetInstance();
init_desc_tbl();
init_runtime_state();
}
void ArrowWorkFlowTest::init_runtime_state() {
ResultQueueMgr* result_queue_mgr = new ResultQueueMgr();
- ThreadResourceMgr* thread_mgr = new ThreadResourceMgr();
- _exec_env._result_queue_mgr = result_queue_mgr;
- _exec_env._thread_mgr = thread_mgr;
+ ThreadResourceMgr* thrbg56gead_mgr = new ThreadResourceMgr();
Review comment:
this spelling mistake will case test case exec failed.
----------------------------------------------------------------
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, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [incubator-doris] morningman merged pull request #2970: fix unit
test randomly failed
Posted by GitBox <gi...@apache.org>.
morningman merged pull request #2970: fix unit test randomly failed
URL: https://github.com/apache/incubator-doris/pull/2970
----------------------------------------------------------------
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, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org