Discussion :

[Donpi]

Bonjour,

Tout d'abord, désolé pour le titre, je ne sais pas quoi mettre comme mot clé parlant (si qqun à une idée, je change volontiers).

Donc, je reprend aujourd'hui une procédure stockée développée par un consultant qui fait des checkDB sur toutes nos bases.
Le problème est que la procédure plante car l'une de nos db à un filegroup en lecture seule.

Du coup j'ai ajouté un test pour savoir si la DB à filegroup en readonly et passer en single user au besoin pour faire le checkdb.
ça fonctionne bien sauf pour master.

Même si le système n’exécutera jamais le code, il me retourne quand même une erreur.

J'ai isolé le cas dans le code ci-dessous.

Quelque 'un a une idée pour contourner le problème ?
L'approche simple serait d'ignorer master en amont mais je me demande si il n'y a pas une autre approche.
Il faut imaginer que tout ça est prix dans une boucle avec du Sql dynamique.

A+

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
 
--IF EXISTS(Select Top 1 1 from master.sys.filegroups Where is_read_only=1)
IF 1=2
BEGIN
	ALTER DATABASE master SET SINGLE_USER WITH ROLLBACK IMMEDIATE
END
ELSE
BEGIN 
	print 'Voilà on doit pas faire de alter table'
END

[aieeeuuuuu]

Bonjour,

L'erreur intervient donc au moment du parsing et non à l'exécution.

Pour contourner le probléme, on peut passer par un EXEC, qui ne sera parsé qu'au moment de l'exécution, si elle a lieu :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
IF 1=2
BEGIN
	EXEC('ALTER DATABASE master SET SINGLE_USER WITH ROLLBACK IMMEDIATE')
END
ELSE
BEGIN 
	print 'Voilà on doit pas faire de alter table'
END

[Donpi]

...
Pour contourner le probléme, on peut passer par un EXEC, qui ne sera parsé qu'au moment de l'exécution
...

Le problème est que le tout est déjà dans un Exec.

Cela dit, je nais pas essayé de faire un exec dans un exec. J'y vais de ce pas.

Edit : alors c'est le festival des quotes mais ça fait ce que ça doit.

[aieeeuuuuu]

oui, ça doit pas faciliter la maintenance

Pas possible de traiter le cas "master" en amont, au moment de la construction des instructions dynamiques pour éviter l'imbrication des EXEC ?

Sinon, une alternative, si ça peut donner une piste :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
 
 
use master;
IF 1=2
BEGIN
	ALTER DATABASE CURRENT SET SINGLE_USER WITH ROLLBACK IMMEDIATE;
END
ELSE
BEGIN 
	print 'Voilà on doit pas faire de alter table'
END

ça, ça ne plante pas.

[Donpi]

Effectivement, pour bien je vais les traiter en amont.
Au final j'ai trop peur d'une fausse manip ou d'un bug je ne sais ou qui me change les statut des DB System.

Du coup j'ai mis un IF pour exclure les db système.