Skip to content
This repository was archived by the owner on Jan 17, 2025. It is now read-only.

Add SESSION TIMEOUT support and missing defer rows.Close()#101

Merged
robertomczak merged 2 commits intomasterfrom
AI-1698
Dec 27, 2022
Merged

Add SESSION TIMEOUT support and missing defer rows.Close()#101
robertomczak merged 2 commits intomasterfrom
AI-1698

Conversation

@robertomczak
Copy link
Copy Markdown

@robertomczak robertomczak commented Dec 23, 2022

This PR

  • adds support of SESSION TIMEOUT user settinghttps://docs.aws.amazon.com/redshift/latest/dg/r_ALTER_USER.html

  • adds missing rows.Close() in order to avoid condition where connection is kept open and idle due to not closed result set.
    With larger set of in this case grants it could result with threads waiting for result set cleanup that never occurred.

@robertomczak robertomczak marked this pull request as draft December 23, 2022 15:40
@robertomczak robertomczak changed the title Add missing defer rows.Close() Add SESSION TIMEOUT support and missing defer rows.Close() Dec 27, 2022
@robertomczak robertomczak marked this pull request as ready for review December 27, 2022 13:21
@robertomczak robertomczak merged commit 16afef4 into master Dec 27, 2022
@robertomczak robertomczak deleted the AI-1698 branch December 27, 2022 13:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants