Kharec/goyco
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: remove redundat getbyemail call to reduce db query by 2 (1Q/user creation instead of 2)

Browse Source
This commit is contained in:
Sandro CURY CAZZANIGA
2025-11-21 16:43:46 +01:00
parent fa9474d863
commit 39598a166d
2 changed files with 2 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
cmd/goyco/commands/parallel_processor.go
View File

@@ -352,12 +352,7 @@ func (p *ParallelProcessor) createSingleUser(userRepo repositories.UserRepositor
const maxRetries = 10 const maxRetries = 10
for range maxRetries { for range maxRetries {
user, err := userRepo.GetByEmail(email) user := &database.User{
if err == nil {
return *user, nil
}
user = &database.User{
Username: username, Username: username,
Email: email, Email: email,
Password: p.passwordHash, Password: p.passwordHash,

7
cmd/goyco/commands/seed.go
View File

@@ -269,12 +269,7 @@ func ensureSeedUser(userRepo repositories.UserRepository, passwordHash string) (
const maxRetries = 10 const maxRetries = 10
for range maxRetries { for range maxRetries {
user, err := userRepo.GetByEmail(seedEmail) user := &database.User{
if err == nil {
return user, nil
}
user = &database.User{
Username: seedUsername, Username: seedUsername,
Email: seedEmail, Email: seedEmail,
Password: passwordHash, Password: passwordHash,