Re: [DISCUSS] Add requested CI check for Pull Requests

2022-04-25 Thread Lijia Liu
g<dev@doris.apache.org> 抄送至 主题 [DIS

Re:Re: [DISCUSS] Add requested CI check for Pull Requests

2022-04-25 Thread 陈明雨
> BTW, I have a small problem, currently BE UT, BE UT, P0 regression, Incubator > Doris UT are all running with personal accounts, the risk of whether they can > ? > continue to run, and also need the account password to view the failure > information, so it is not very convenient, can you migra

Re:回复:[DISCUSS] Add requested CI check for Pull Requests

2022-04-25 Thread 陈明雨
"dev" > 发送时间: 2022年4月25日(星期一) 下午4:13 >收件人: "dev" >主题: [DISCUSS] Add requested CI check for Pull Requests > > > >Hello everyone, with the increasing num

Re: [DISCUSS] Add requested CI check for Pull Requests

2022-04-25 Thread De Li
+1 李德 l...@apache.org On 2022/04/25 08:13:20 GuoLei Yi wrote: > Hello everyone, with the increasing number of pull requests, we need to > strictly control the code merge process. I plan to turn on "requested" CI > checks, that is, when all "requested" CI checks of a PR pass, the code can > be mer

Re:[DISCUSS] Add requested CI check for Pull Requests

2022-04-25 Thread stalary
+1 | | stalary | | stal...@163.com | 签名由网易邮箱大师定制 On 04/25/2022 16:13,GuoLei Yi wrote: Hello everyone, with the increasing number of pull requests, we need to strictly control the code merge process. I plan to turn on "requested" CI checks, that is, when all "requested" CI checks of a PR pass, t

Re: [DISCUSS] Add requested CI check for Pull Requests

2022-04-25 Thread 41108453
+1 -- Original -- From: GuoLei Yi

Re: [DISCUSS] Add requested CI check for Pull Requests

2022-04-25 Thread ling miao
> BTW, I have a small problem, currently BE UT, BE UT, P0 regression, Incubator Doris UT are all running with personal accounts, the risk of whether they can continue to run, and also need the account password to view the failure information, so it is not very convenient, can you migrate to the off

Re: [DISCUSS] Add requested CI check for Pull Requests

2022-04-25 Thread zhg yang
+1 BTW, I have a small problem, currently BE UT, BE UT, P0 regression, Incubator Doris UT are all running with personal accounts, the risk of whether they can continue to run, and also need the account password to view the failure information, so it is not very convenient, can you migrate to th

??????[DISCUSS] Add requested CI check for Pull Requests

2022-04-25 Thread 952130278
Maybe we should add a test to confirm clang compatibility? Some codes cannot be compiled with clang, but can be compiled with gcc. -- -- ??:

Re:Re: [DISCUSS] Add requested CI check for Pull Requests

2022-04-25 Thread 陈明雨
> 4. I can not login to the new TeamCity. The CI tools should easily accessible to all contributors. You can click "login as guest" And later I think the PPMC member should have admin privilege. -- 此致!Best Regards 陈明雨 Mingyu Chen Email: chenmin...@apache.org 在 2022-04-25 16:30:31,"ling mia

??????[DISCUSS] Add requested CI check for Pull Requests

2022-04-25 Thread stephen
+1 --  -- ??: "dev"

Re: [DISCUSS] Add requested CI check for Pull Requests

2022-04-25 Thread ling miao
1. FE code format should be "requested" in the future. 2. There are some duplicate checks should be removed such as: Incubator Doris UT. 3. I recommend keeping the original P0 check until the P0 case is migrated. 4. I can not login to the new TeamCity. The CI tools should easily accessible to al

Re:[DISCUSS] Add requested CI check for Pull Requests

2022-04-25 Thread 陈明雨
+1 -- 此致!Best Regards 陈明雨 Mingyu Chen Email: chenmin...@apache.org At 2022-04-25 16:13:20, "GuoLei Yi" wrote: >Hello everyone, with the increasing number of pull requests, we need to >strictly control the code merge process. I plan to turn on "requested" CI >checks, that is, when all "re

[DISCUSS] Add requested CI check for Pull Requests

2022-04-25 Thread GuoLei Yi
Hello everyone, with the increasing number of pull requests, we need to strictly control the code merge process. I plan to turn on "requested" CI checks, that is, when all "requested" CI checks of a PR pass, the code can be merged. The following admissions will be forced on: 1. BE Code Formatter /