Skip to content

Conversation

@kyleconroy
Copy link
Collaborator

Corrected "Undertand" to "Understand" in the TODO comment for the CreateFunctionStmt struct fields.

🤖 Generated with Claude Code

Corrected "Undertand" to "Understand" in the TODO comment for the
CreateFunctionStmt struct fields.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. 🔧 golang labels Oct 20, 2025
Fixed a race condition in the checker's Client field initialization
by using sync.Once to ensure thread-safe, single initialization even
when fetchDatabaseUri is called concurrently.

Changes:
- Added sync.Once field (clientOnce) to checker struct
- Replaced check-then-act pattern with sync.Once.Do()
- Removed FIXME comment as issue is now resolved

This ensures that even if multiple goroutines call fetchDatabaseUri
simultaneously, the Client will only be initialized once without
any race conditions.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Oct 20, 2025
@kyleconroy kyleconroy merged commit 92e5a24 into main Oct 21, 2025
13 checks passed
@kyleconroy kyleconroy deleted the claude/setup-test-runner-011CUK2Zagac4BKVvEU3vkhi branch October 21, 2025 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files. 🔧 golang

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants