Skip to content

More optimization work to eliminate timeouts#55

Merged
jmgasper merged 2 commits intomasterfrom
develop
Feb 26, 2026
Merged

More optimization work to eliminate timeouts#55
jmgasper merged 2 commits intomasterfrom
develop

Conversation

@jmgasper
Copy link
Collaborator

No description provided.

@jmgasper jmgasper merged commit d10de4e into master Feb 26, 2026
5 of 6 checks passed
LIMIT 1
),
filtered_challenges AS (
filtered_challenges AS MATERIALIZED (

Choose a reason for hiding this comment

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

[⚠️ performance]
Consider evaluating if the MATERIALIZED keyword is necessary for the filtered_challenges CTE. While it can improve performance by caching the result, it may also increase memory usage. Ensure this trade-off is beneficial for your use case.

)
),
registrants AS (
registrants AS MATERIALIZED (

Choose a reason for hiding this comment

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

[⚠️ performance]
The MATERIALIZED keyword is used for the registrants CTE. Verify if materializing this CTE provides a significant performance benefit, as it can increase memory usage. Consider removing it if the performance gain is negligible.

END AS "challengeCompletedDate",
r."registrantHandle",
sub."registrantFinalScore"
FROM registrants r

Choose a reason for hiding this comment

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

[⚠️ performance]
The LEFT JOIN LATERAL for winnerHandle and registrantFinalScore could potentially be optimized if these subqueries are expensive. Consider evaluating their performance impact, especially if the dataset is large.

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.

1 participant