Promote dev to main: Updates_20260601#800
Merged
Merged
Conversation
- Add @encryption_list parameter (0, 1, or 0,1 for both) - Pre-compilation ALTER TABLE guard adds encryption column to existing backup_performance_results tables before ALTER PROCEDURE compiles, preventing Msg 207 invalid column name at compile time - CREATE TABLE inside proc body updated to include encryption column on fresh installs, matching original pattern - DMK and certificate pre-validation before test loop with descriptive RAISERROR messages using @msg variable pattern - Certificate name resolved once before loop, not per iteration - encryption threaded through cursor, FETCH, progress messages, BACKUP WITH clause (AES_256), and INSERT INTO results table - encryption column added to all 5 result sets, GROUP BY, PARTITION BY - Result set 2 now partitions on compression+encryption pairing - Result set 3 adds encryption UNION ALL block for parameter impact - Version bumped to 1.1, version_date 20260518 - Tested on SQL Server 2022 CU24, confirmed AES_256 encryption via msdb.dbo.backupset encryptor_type and thumbprint verification
…-parameter Add @encryption_list parameter for backup encryption testing
- sp_IndexCleanup 2.6 -> 2.7 - sp_QuickieStore 6.6 -> 6.7 - sp_HumanEvents 7.6 -> 7.7 - TestBackupPerformance 1.0 -> 1.1 (new @encryption_list feature) - ProtectSession 1.0 (date only; first public release) - All @version_date -> 20260601 - ProtectSession: add EXAMPLE CALLS result set to @help = 1 output (basic / aggressive / defensive invocations) to match the help UX pattern from sp_HumanEvents Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- sp_IndexCleanup: 11 nvarchar(MAX) -> nvarchar(max) (params, locals, temp-table columns, dynamic-SQL CONVERTs, xml.value literal) - sp_QuickieStore: 2 nvarchar(MAX) -> nvarchar(max) on @include_databases / @exclude_databases parameters - sp_HumanEvents: 10 CAST() -> CONVERT() with arg order swapped (8 inline string-concat sites for blocking/blocked spid + ecid, 2 multi-line CASTs in the blocking-chain recursive CTE); TRY_CAST sites preserved per style guide (TRY_CAST is allowed) No behavior change. Pre-existing style debt flagged in the Updates_20260601 release audit; carved out from the version-bump PR so the release diff stays tight. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Updates_20260601 release prep
…0601 Style sweep: lowercase nvarchar(max), CAST -> CONVERT
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Cuts the
Updates_20260601release. Six work PRs merged intodevsinceUpdates_20260501(plus one direct commit for ProtectSession), all rolled up here for promotion tomain.Procedure changes
HAS_DBACCESSpreflight to avoid hang on databases the caller can't access'failed'to@execution_type_desc(bundlesaborted+exception); #793: fixplan_count/query_countunder-count in@find_high_impact@output_schema_nameis notdbo; #787: remove strayAS sqlhandlefrom blocking-tableINSERT@encryption_listparameter for backup encryption testing (thanks @bfountain-1976)Also rolled up
@help = 1example callsnvarchar(MAX)->nvarchar(max),CAST->CONVERT)Test plan
@help = 1returns intro + parameters + EXAMPLE CALLS + license result setsdbo.backup_performance_results) creates the table with theencryptioncolumndbo.backup_performance_resultsgets theencryptioncolumn added via theCOL_LENGTHguard beforeALTER PROCEDUREcompilessort_orderrecursive CTE still produces the same chain strings after theCAST->CONVERTswap🤖 Generated with Claude Code