Re-evaluate performances for []*Struct over []Struct
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Refactoring
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- go
- Ambito
- performance, testing-qa
Direzione di ricerca
Inizia leggendo gli utilizzi elencati in pkg/test/fetcher.go, pkg/job/reporter.go, pkg/job/job.go, pkg/runner/job_runner.go, pkg/jobmanager/jobmanager.go e nei test fetcher URI e literal. Confronta i casi []Struct e []*Struct, inclusi Targets, e valuta ogni rappresentazione in modo coerente. Il lavoro è completato quando ogni caso elencato è stato rivalutato e tutte le modifiche giustificate sono state applicate nei file interessati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Wednesday Feb 05, 2020 at 12:48 GMT
Originally opened as https://github.com/facebookincubator/contest/issues/21
Using []Struct is in most cases probably desirable, but in some cases []*Struct might make sense (e.g. Targets). We should re-evaluate these cases. Here is where we use []*, excluding Targets:
pkg/test/fetcher.go:16: Fetch(interface{}) (string, []*TestStepDescriptor, error)
pkg/job/reporter.go:20: Report(cancel <-chan struct{}, parameters interface{}, results []*test.TestResult, ev testevent.Fetcher) (bool, interface{}, error)
pkg/job/job.go:27: TestDescriptors []*test.TestDescriptor
pkg/job/job.go:63: Tests []*test.Test
pkg/runner/job_runner.go:49: testResults []*test.TestResult
pkg/jobmanager/jobmanager.go:88: tests := make([]*test.Test, 0, len(jd.TestDescriptors))
plugins/testfetchers/uri/uri.go:86:func (tf *URI) Fetch(params interface{}) (string, []*test.TestStepDescriptor, error) {
plugins/testfetchers/uri/uri.go:117: Steps []*test.TestStepDescriptor
plugins/testfetchers/literal/literal.go:29: Steps []*test.TestStepDescriptor
plugins/testfetchers/literal/literal.go:54:func (tf *Literal) Fetch(params interface{}) (string, []*test.TestStepDescriptor, error)
- Lingua principale
- Go
- Stelle
- 20
- Fork
- 17
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
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.
Altre issue di linuxboot/contest
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
-
update readmeForse già presa @mimir-d l’ha presa 1252 giorni fa. Apertabug documentation
-
enhancement good first issue
Difficoltà 3/5 1-2 giorni Idoneità per principianti 38/100
-
enhancement good first issue v2_overhaul
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
-
enhancement good first issue
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
Tutte le issue di linuxboot/contest
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
duplication
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
openvibely/openvibely#1443 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 60/100
canonical/service-mesh#845 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 80/100
keyxmakerx/Chronicle#1179 ·
I maintainer di solito rispondono entro 1 giorno