פתיחת מסד נתונים בכל קריאה

[יוסף בן שמעון]

@dovid אכן, עכשיו אני מבין שנכנסתי לסוגיא בראש של נוד, אם אתה עושה close אתה הורג את המאגר, צחוק הגורל שדוטנט השפה הכי עילית שרק עושים פווו ויש לך אפליקציה, דוקא הם לא טרחו להסתיר ממך את המימוש של סגירת חיבור והחזרתו למאגר, ואילו בנוד ובגו שלכאורה הרבה יותר "ברזליות", המימוש הזה מוסתר מעיני המפתח

[nigun]

@יוסף-בן-שמעון
אז מה אני אמור לעשות?

1. ליצור משתנה גלובלי
   2.לבצע פתיחה רק פעם אחת

[dovid]

@nigun תביא לי אתר שמסביר איך להשתמש בsql (עם דוגמאות עשירות ומפורטות ולא תיעוד) ואני אכתוב לך בדיוק מה לעשות.

[nigun]

@dovid
אלו טובים?
http://go-database-sql.org/accessing.html
https://www.alexedwards.net/blog/organising-database-access

[dovid]

@nigun כן.
לפי מה שכתוב לך אין לך בכלל פתיחת קונקשיין.
הdb.Open לא יוצר קונקשיין אלא סה"כ מקים מופע שנועד מטבעו להיות בעל חיים ארוכים ולא אמורים לסגרו (שם הוא סוגר אותו עם defer כשהתוכנית תיגמר). המופע הזה יכול להיות אחד לכל התוכנה כמו ש@יוסף-בן-שמעון אמר.
מה עם קונקשנים? זה נראה שמנהלים לך את זה מצויין מאחורי הקלעים (יש לזה חיסרון קצת כי אם אני מבין טוב אתה לא יכול להבטיח ששתי פקודות ירוצו על אותו קונקשיין) ובעצם כל קריאה לQuery מחזיקה קונקשיין שנסגר בrows.Close().
אז סיכמנו: var db *sql.DB אחד לכל האפליקציה.

[nigun]

שוב למדתי בדרך הקשה שצריך לקרוא דוקומנטיצות מהתחלה לסוף
בגלל שלא ידעתי בדיוק איך מתנהל הסשנים בGO
הגעתי מהר מאוד לSQL Error 1040: Too Many Connection
אחרי שקראתי את המדריך שהבאתי למעלה הייתי צריך לעשות שינויים גדולים בקוד בשביל לתקן את זה
(מזל שהרצתי בדיקת עומסים)

[יוסף בן שמעון]

@nigun פרט הרחב נמק

[nigun]

@יוסף-בן-שמעון
א. קראתי לבקשת INSERT עם Query ולא עם exec
כיוון שאני לא צריך את הפלט זה מבזבז נתונים, כי הוא יוצר פלט וזה משאיר את החיבור פתוח ליותר זמן
ב. במקרה של שגיאה עשיתי return בלי פקודה נוספת לrows.Close()
מה שגרם לחלק מהבקשות להישאר פתוח משום מה.
ולמדתי שמומלץ לזרוק הרבה Close כי אם לא יועיל לא יזיק
ג. למדתי שעדיף באופן כללי להכין את הבקשה עם Prepare ולעשות עליו כמה בקשות
(במקרה שלי כל בקשה נמצא בתוך פונקציה נפרדת כך שוויתרתי על זה כי זה מידי הרבה עבודה לבנות את כל הפונקציות מחדש)
ד. למדתי שאם בקשה אמורה לקבל רק שורה אחת משתמשים בQueryRow ולא בQuery
יש עוד כמה דברים שלא ירדתי לעומקם עדיין למשל ניהול ערכים רקים (NULL)

[dovid]

א. נראה לי שגוי לגמרי. הפלט הוא מספר קטן של 32 סיביות שזה כלום שבכלום, ובכל מקרה הקוד שלך מחכה לסיום הביצוע. המתאים בזה הוא QueryRow שהבאת בהמשך.
ב. תמיד תעשה defer rows.Close() שורה מתחת היצירה שלהם. ככה אין מצב שתפספס את הסגירה ותוותר לגמרי על הזריקה של הClose בכל מקום.

[nigun]

@dovid
אני לא יודע למה אבל כשהרצי מאות בקשות בלי לזרוק סגירה בכל מקום נתקעתי עם Too Many Connection
ועכשיו זה כבר לא קורה