So sorry for discussing in time. I think your mean is the `state` of the user is restricted to verify whether a user has permission to login.
What I expressed int the last email is the following means. Because the user has not state before,so the sql of query the datasources、resources、process definitons、projects needn't filter `state` field. Now if add the `state`,the above query functions whether need add `where state=?` ? If one user created all datasources,resources and projects. Whether the state` of a user change the avaiablility of all things one user created? If it will change,We will add `where state=?` in all query functions. If it will not change,we will not change the query functions. DolphinScheduler(Incubator) PPMC Gang Li 李岗 [email protected]<mailto:[email protected]> 发件人： wenhemin 发送时间： 2020-05-05 08:13 收件人： dev 主题： Re: [DISCUSS] Add "state" field to table t_ds_user First, the purpose of disabling the user is that the user does not have permission to log in to the system. 1. If it was created by the user, the user cannot be used after being disabled. If the permissions have been assigned to other users, there should be an admin operation to revoke the permissions. I think, in business, most processes should not be automatically disabled or deleted. 2. After the user is disabled. Because you can no longer log in to the system. So, you can't create a project. > 在 2020年5月2日，21:42，[email protected] 写道： > > > Great,I think we can discuss the other aspect. If add the `state` in table > t_ds_user.It will also change other places. > Such as the following. > 1、If a user created the process definition or resource or datasource,We > change the user state from enable to disable,the resource or datasource > should also disable. > 2、If a user is disable,Whether it can create the resource or datasource? > > > DolphinScheduler(Incubator) PPMC > Gang Li 李岗 > > [email protected]<mailto:[email protected]> > > 发件人： wenhemin > 发送时间： 2020-04-26 11:23 > 收件人： dev > 主题： [DISCUSS] Add "state" field to table t_ds_user > hello, friends! > > The issue: https://github.com/apache/incubator-dolphinscheduler/issues/1923 > <https://github.com/apache/incubator-dolphinscheduler/issues/1923> > Need to add a “state” field, Control whether users can log in to the system. > > Sql : ALTER TABLE t_ds_user ADD `state` int(1) DEFAULT 1 COMMENT 'state > 0:disable 1:enable'; > > The PR: https://github.com/apache/incubator-dolphinscheduler/pull/2424 > <https://github.com/apache/incubator-dolphinscheduler/pull/2424>
