Added connection timeout. Fixes #7417 #7438
Conversation
Added splatting fof command for readability
| @@ -367,7 +367,7 @@ function Invoke-DbaDbDataMasking { | |||
|
|
|||
| $query = "CREATE NONCLUSTERED INDEX [$($maskingIndexName)] ON [$($dbTable.Schema)].[$($dbTable.Name)]([$($identityColumn)])" | |||
|
|
|||
| Invoke-DbaQuery -SqlInstance $server -SqlCredential $SqlCredential -Database $db.Name -Query $query | |||
| Invoke-DbaQuery -SqlInstance $server -SqlCredential $SqlCredential -Database $db.Name -Query $query -QueryTimeout | |||
There was a problem hiding this comment.
Is there a $ConnectionTimeout missing at the end?
There was a problem hiding this comment.
Or better $CommandTimeout?
There was a problem hiding this comment.
This is probably the reason the tests fail.
|
Why do you use |
Formatted list for better readability
Splatted command Replaced connection timeout with command timeout
|
@andreasjordan You are correct, there was an obsolete parameter. |
|
🙌🏼 awesome, thank you both! |
Added splatting fof command for readability
Type of Change
.\tests\manual.pester.ps1)Purpose
The invoke command the connection timeout to make sure that commands would not execute for a long period of time
Approach
Added variable to several query commands
Commands to test
Invoke-DbaDbDataMasking