Default to ignoring Temp Tables for Compression check

[lowlydba]

I believe in most cases it isn't beneficial to mandate compression for temporary tables.

tsqllint/source/TSQLLint.Infrastructure/Rules/DataCompressionOptionRule.cs

Lines 24 to 33 in cfa67da

	public override void Visit(CreateTableStatement node)
	{
	var childCompressionVisitor = new ChildCompressionVisitor();
	node.AcceptChildren(childCompressionVisitor);
	if (!childCompressionVisitor.CompressionOptionExists)
	{
	errorCallback(RULE_NAME, RULE_TEXT, node.StartLine, GetColumnNumber(node));
	}
	}

Requesting that this not apply at all to temp tables, or be configurable with a default of not applying to temp tables.

[nathan-boyd]

This will be tricky to handle. Changing the rule would be a breaking change for existing users, creating sub-config for each rule is an option, the schema doesn't currently support it but it wouldn't be too major of a change.

[lowlydba]

Yeah, the breaking change aspect does make it a bit more nuanced.

This is probably a low-priority change, but might be useful to integrate for new rules that create/alter tables as they come up. For now we'll probably add guidance on using the exclusion comments for these scenarios.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.