matomo-org / matomo-org/plugin-CustomVariables
Should we limit key values to varchar(50)?
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 4
- Forks
- 6
- Ø Merge
- 6 Std. 18 Min.
- Gemergte PRs (30 T.)
- 6
Beschreibung
When running
core:convert-to-utf8mb4
I got for table log_visit:
[Zend_Db_Adapter_Exception]
SQLSTATE[42000]: Syntax error or access violation: 1118 Row size too large. The maximum row size for the used table type, not counting BLOBs, is 65535. This includes storage overhead,
check the manual. You have to change some columns to TEXT or BLOBs
My setup has 20 custom variables and 15 custom dimensions in scope visit.
To fix this, I decided to convert some of the custom_var_k* columns to varchar(100) as 100 chars for the key still seem to be long enough.
I think this change would make sense for 99% of matomo users and would give an easier upgrade path for big Matomo 3.x setups
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit dem Kern: dem Befehl core:convert-to-utf8mb4 und dem Schema log_visit, wobei du dich auf die im Bericht beschriebenen Spalten custom_var_k* konzentrierst. Ermittle, ob eine Änderung ihrer Schlüssellänge den row-size-Fehler vermeidet und gleichzeitig bestehende Upgrades beibehält, und überprüfe, dass die Konvertierung bei Setups mit vielen benutzerdefinierten Variablen und Dimensionen erfolgreich ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- databases
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 42/100