DROP USER занимает слишком много времени, когда много пользователей

11

В экземпляре SQL Server 2014 с достаточным количеством оперативной памяти и быстрыми дисками более 160 пользователей имеют доступ к базе данных. По какой-то причине без моего ведома выполнение команды DROP USER [username]в этой базе данных занимает до 5 секунд на пользователя.

Преобразование пользователей в логины и восстановление их разрешений происходит очень быстро.

В контексте обновления баз данных DEV с производства я должен отбросить и воссоздать всех пользователей баз данных. Так что да, удаление пользователей базы данных и их воссоздание необходимо.

Как мне ускорить DROP USERкоманду?

Помните, я должен запустить его более 160 раз для экземпляра, о котором я пишу.

Это SQL, который я использую:

DECLARE drop_user_cur CURSOR FOR 
SELECT name FROM #drop_users

OPEN drop_user_cur
FETCH NEXT FROM drop_user_cur INTO @user

WHILE @@FETCH_STATUS = 0
BEGIN
    SET @sql = 'use [' + @db_name + '] DROP USER [' + @user + ']'
    BEGIN TRY
        print @sql
        EXECUTE(@sql)
    END TRY
    BEGIN CATCH
        print 'ERREUR : ' + @sql
    END CATCH
    FETCH NEXT FROM drop_user_cur INTO @user
END

CLOSE drop_user_cur
DEALLOCATE drop_user_cur

Проблема не в курсоре; это факт, DROP USERкоторый занимает до 5 секунд.

Используя sp_whoisactive, wait_type есть NULL.

введите описание изображения здесь

Не обращать внимание на продолжительность, DROPи CREATE USERбыли бежать в WHILEпетлю из -за чего он больше минуты говорят.

Профилировщик показывает более 125 000 операций чтения для выполнения DROP USER.

Компонент Service Broker не включен.

Крейг Эфрейн
источник
1
@CraigEfrein Вы можете восстановить свой ответ. Выглядит хорошо, и спасибо, что включили мой комментарий в ваш ответ. Я рад, что вы нашли решение работающим!
Кин Шах

Ответы:

6

Решением этой проблемы было включение сервисного брокера в базу данных.

ALTER DATABASE [Database_name] SET NEW_BROKER WITH ROLLBACK IMMEDIATE;

После включения сервис-брокера для базы данных, отбрасывание пользователей происходило практически мгновенно.

Кин спросил, был ли сервисный брокер включен в предыдущем комментарии, который отправил мне поиск в правильном направлении.

Крейг Эфрейн
источник
2

это не ответ на вопрос, а аргумент, чтобы полностью отклонить его.

Если я правильно понимаю ваш комментарий:

В контексте обновления баз данных DEV с производства я должен отбросить и воссоздать всех пользователей баз данных. Так что да, удаление пользователей базы данных и их воссоздание необходимо.

Ваша цель - скопировать производственные данные в систему разработки и заставить их работать с тем же разрешением, что и у производственной системы, используя те же имена пользователей .

Самый быстрый путь - клонировать sql-логины из производственной системы в систему dev, используя процедуру, описанную ms в этой статье kb .

с помощью процедуры, определенной ms, вы можете восстановить базы данных на своем dev-сервере, и разрешение будет работать без изменений.

Я успешно использовал вышеупомянутую процедуру для копирования рабочей среды sql 2005 в среду test и dev sql 2012.
После клонирования пользователей простое восстановление базы данных привело меня к паре полностью работающих новых систем с актуальными данными.

Огромный плюс этого решения заключается в том, что для обновления данных разработчика вы просто восстанавливаете производственную базу данных и все; вам придется настроить ссылки, синонимы и тому подобное, но разрешения не будут нарушены.

Вот код из этой статьи, просто для справки, но, пожалуйста, обратитесь к статье, в которой есть замечания и подробности о совместимости с древними версиями sql, сведениями о шифровании паролей и другой информацией, которая может избавить вас от головной боли при передаче имен входа на серверы:

USE master
GO
IF OBJECT_ID ('sp_hexadecimal') IS NOT NULL
  DROP PROCEDURE sp_hexadecimal
GO
CREATE PROCEDURE sp_hexadecimal
    @binvalue varbinary(256),
    @hexvalue varchar (514) OUTPUT
AS
DECLARE @charvalue varchar (514)
DECLARE @i int
DECLARE @length int
DECLARE @hexstring char(16)
SELECT @charvalue = '0x'
SELECT @i = 1
SELECT @length = DATALENGTH (@binvalue)
SELECT @hexstring = '0123456789ABCDEF'
WHILE (@i <= @length)
BEGIN
  DECLARE @tempint int
  DECLARE @firstint int
  DECLARE @secondint int
  SELECT @tempint = CONVERT(int, SUBSTRING(@binvalue,@i,1))
  SELECT @firstint = FLOOR(@tempint/16)
  SELECT @secondint = @tempint - (@firstint*16)
  SELECT @charvalue = @charvalue +
    SUBSTRING(@hexstring, @firstint+1, 1) +
    SUBSTRING(@hexstring, @secondint+1, 1)
  SELECT @i = @i + 1
END

SELECT @hexvalue = @charvalue
GO

IF OBJECT_ID ('sp_help_revlogin') IS NOT NULL
  DROP PROCEDURE sp_help_revlogin
GO
CREATE PROCEDURE sp_help_revlogin @login_name sysname = NULL AS
DECLARE @name sysname
DECLARE @type varchar (1)
DECLARE @hasaccess int
DECLARE @denylogin int
DECLARE @is_disabled int
DECLARE @PWD_varbinary  varbinary (256)
DECLARE @PWD_string  varchar (514)
DECLARE @SID_varbinary varbinary (85)
DECLARE @SID_string varchar (514)
DECLARE @tmpstr  varchar (1024)
DECLARE @is_policy_checked varchar (3)
DECLARE @is_expiration_checked varchar (3)

DECLARE @defaultdb sysname

IF (@login_name IS NULL)
  DECLARE login_curs CURSOR FOR

      SELECT p.sid, p.name, p.type, p.is_disabled, p.default_database_name, l.hasaccess, l.denylogin FROM 
sys.server_principals p LEFT JOIN sys.syslogins l
      ON ( l.name = p.name ) WHERE p.type IN ( 'S', 'G', 'U' ) AND p.name <> 'sa'
ELSE
  DECLARE login_curs CURSOR FOR


      SELECT p.sid, p.name, p.type, p.is_disabled, p.default_database_name, l.hasaccess, l.denylogin FROM 
sys.server_principals p LEFT JOIN sys.syslogins l
      ON ( l.name = p.name ) WHERE p.type IN ( 'S', 'G', 'U' ) AND p.name = @login_name
OPEN login_curs

FETCH NEXT FROM login_curs INTO @SID_varbinary, @name, @type, @is_disabled, @defaultdb, @hasaccess, @denylogin
IF (@@fetch_status = -1)
BEGIN
  PRINT 'No login(s) found.'
  CLOSE login_curs
  DEALLOCATE login_curs
  RETURN -1
END
SET @tmpstr = '/* sp_help_revlogin script '
PRINT @tmpstr
SET @tmpstr = '** Generated ' + CONVERT (varchar, GETDATE()) + ' on ' + @@SERVERNAME + ' */'
PRINT @tmpstr
PRINT ''
WHILE (@@fetch_status <> -1)
BEGIN
  IF (@@fetch_status <> -2)
  BEGIN
    PRINT ''
    SET @tmpstr = '-- Login: ' + @name
    PRINT @tmpstr
    IF (@type IN ( 'G', 'U'))
    BEGIN -- NT authenticated account/group

      SET @tmpstr = 'CREATE LOGIN ' + QUOTENAME( @name ) + ' FROM WINDOWS WITH DEFAULT_DATABASE = [' + @defaultdb + ']'
    END
    ELSE BEGIN -- SQL Server authentication
        -- obtain password and sid
            SET @PWD_varbinary = CAST( LOGINPROPERTY( @name, 'PasswordHash' ) AS varbinary (256) )
        EXEC sp_hexadecimal @PWD_varbinary, @PWD_string OUT
        EXEC sp_hexadecimal @SID_varbinary,@SID_string OUT

        -- obtain password policy state
        SELECT @is_policy_checked = CASE is_policy_checked WHEN 1 THEN 'ON' WHEN 0 THEN 'OFF' ELSE NULL END FROM sys.sql_logins WHERE name = @name
        SELECT @is_expiration_checked = CASE is_expiration_checked WHEN 1 THEN 'ON' WHEN 0 THEN 'OFF' ELSE NULL END FROM sys.sql_logins WHERE name = @name

            SET @tmpstr = 'CREATE LOGIN ' + QUOTENAME( @name ) + ' WITH PASSWORD = ' + @PWD_string + ' HASHED, SID = ' + @SID_string + ', DEFAULT_DATABASE = [' + @defaultdb + ']'

        IF ( @is_policy_checked IS NOT NULL )
        BEGIN
          SET @tmpstr = @tmpstr + ', CHECK_POLICY = ' + @is_policy_checked
        END
        IF ( @is_expiration_checked IS NOT NULL )
        BEGIN
          SET @tmpstr = @tmpstr + ', CHECK_EXPIRATION = ' + @is_expiration_checked
        END
    END
    IF (@denylogin = 1)
    BEGIN -- login is denied access
      SET @tmpstr = @tmpstr + '; DENY CONNECT SQL TO ' + QUOTENAME( @name )
    END
    ELSE IF (@hasaccess = 0)
    BEGIN -- login exists but does not have access
      SET @tmpstr = @tmpstr + '; REVOKE CONNECT SQL TO ' + QUOTENAME( @name )
    END
    IF (@is_disabled = 1)
    BEGIN -- login is disabled
      SET @tmpstr = @tmpstr + '; ALTER LOGIN ' + QUOTENAME( @name ) + ' DISABLE'
    END
    PRINT @tmpstr
  END

  FETCH NEXT FROM login_curs INTO @SID_varbinary, @name, @type, @is_disabled, @defaultdb, @hasaccess, @denylogin
   END
CLOSE login_curs
DEALLOCATE login_curs
RETURN 0
GO
Paolo
источник
1
Привет. Я не пытаюсь создать точную копию пользователей с производства. DEV на самом деле не является точной копией продукции. При входе в систему не используется один и тот же пароль, а dev не находится в одном домене. Существуют и другие ограничения, которые делают необходимым удаление пользователей базы данных, и я не буду утомлять вас.
Крейг Эфрейн
Что касается безопасности, вы НИКОГДА не должны иметь пароли производства, которые касаются вашей среды разработки. Вы только что предоставили своим разработчикам доступ к производственным ресурсам и надеетесь, что ни у одного из них нет роли администратора безопасности.
если у пользователей есть пароли, то вы имеете дело с пользователями sql, поэтому домен не имеет значения. также пароли не заблокированы, так что вы можете изменить их, как только пользователи будут созданы. скрипт может быть изменен перед его запуском: удалите всех нежелательных / ненужных пользователей, и все будет в порядке. Мое предложение не является решением вашей проблемы, но, возможно, после клонирования вам нужно будет удалить меньше пользователей, потому что разрешение больше не обрабатывается. не оптимально, но возможно улучшение ^^
Паоло
0

Курсор, подобный этому, должен работать

Use DB_name

declare @username varchar(50)
declare user_cursor cursor for select '['+name+']' from sys.database_principals where type in ('u', 's') and principal_id>4
open user_cursor
fetch next from user_cursor into @username
while (@@fetch_status=0)
begin
EXEC sp_dropuser @username
fetch next from user_cursor into @username
end
close user_cursor
deallocate user_cursor
go
Midhun CN
источник