godaddy-wordpress / godaddy-wordpress/wc-plugin-framework
Consider storing jobs without the job ID in the option name
Nessuno ha ancora preso questa issue.
- Lingua principale
- PHP
- Stelle
- 146
- Fork
- 43
- Merge medio
- 21m
- PR unite (30g)
- 3
Descrizione
Since we don't leverage the WP core get|update_option methods any longer, we don't necessarily need the unique job identifier in the option name. This causes issues on some hosts that artificially limit the options table to 64 characters rather than the default 255 since WP 4.4.
We can consider storing the job ID as part of the job data instead, then adjusting our SQL to look for this ID when getting the job since the option names will be duplicated.
Will also need an upgrade in plugins that implement this for the existing job storage. This can be used as a stop-gap until considering whether or not we want to move job storage to a custom table.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia individuando lo storage dei job esistente e l’SQL utilizzato per recuperare i job, insieme ai plugin che implementano questo storage. Esamina come gli ID dei job sono attualmente rappresentati nei nomi delle opzioni e come vengono gestiti gli upgrade. Il lavoro è completato quando i job esistenti rimangono leggibili dopo un upgrade e il nuovo storage funziona con nomi delle opzioni duplicati sugli host che impongono limiti più brevi.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- php, sql, wordpress
- Ambito
- backend, databases
- Tipo di issue
- Refactoring
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100