[GitHub] [incubator-doris] vagetablechicken opened a new issue #5062: [UT] TestEnv needs fix

[GitBox <gi...@apache.org>]

vagetablechicken opened a new issue #5062:
URL: https://github.com/apache/incubator-doris/issues/5062


   It's good to create a TestEnv for unit tests of be. But the current TestEnv can't work. 
   And it doesn't need to be a friend of RuntimeState or any other class, because the unit tests can access private members.
   https://github.com/apache/incubator-doris/blob/6fedf5881be01363d3a246e07c79ffb417574d40/be/src/runtime/runtime_state.h#L365
   
   So I want to move it to the `be/test/` folder, and fix it.


----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org

[GitHub] [incubator-doris] morningman closed issue #5062: [UT] TestEnv needs fix

[GitBox <gi...@apache.org>]

morningman closed issue #5062:
URL: https://github.com/apache/incubator-doris/issues/5062


   


----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org