Skip to content

Purge faster#50

Open
spazmodius wants to merge 4 commits into
masterfrom
pruge_faster
Open

Purge faster#50
spazmodius wants to merge 4 commits into
masterfrom
pruge_faster

Conversation

@spazmodius

Copy link
Copy Markdown
Contributor

Using a temp table for #doomed instead of a table variable @doomed seems to lead to much better performance.
Maybe because of temp table deferred compilation/cardinality sniffing.

@spazmodius spazmodius requested a review from Copilot July 9, 2026 20:02

This comment was marked as low quality.

@spazmodius

Copy link
Copy Markdown
Contributor Author

These changes also need to be ported to https://github.com/versionone/IT-shared/tree/master/demo-data, and maybe backported to prior versions

@spazmodius spazmodius requested a review from Copilot July 9, 2026 20:04

This comment was marked as low quality.

@spazmodius spazmodius requested a review from Copilot July 9, 2026 20:19

This comment was marked as low quality.

@spazmodius spazmodius requested review from lremedi and pmamut July 9, 2026 20:20

@pmamut pmamut left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pmamut pmamut requested a review from Copilot July 9, 2026 21:06

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants