coinbase / coinbase/temporal-ruby
Injecting the metrics and log adapter into the Worker object
- Lingua principale
- Ruby
- Stelle
- 287
- Fork
- 113
- Merge medio
- 6g 11h
- PR unite (30g)
- 1
Descrizione
Hi there,
At the moment, some of the usages of the log and metrics adapters reference the global variables `Temporal.log` and `Temporal.metrics` respectively. This means that the configuration can't be overridden by passing different adapters in the configuration into the `Worker` constructor.
Example: https://github.com/coinbase/temporal-ruby/blob/101ec13947a0c2c9bc1754de87aeaf7c697fb54f/lib/temporal/activity/task_processor.rb#L28-L29
I've noticed that there are more and more things are starting to be injected via the `Worker`. Is doing the same thing with these adapters possible? Would you accept a change which added that?
Thanks!
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Inizia da lib/temporal/activity/task_processor.rb alle righe indicate, quindi traccia gli altri utilizzi di Temporal.log e Temporal.metrics e il modo in cui gli adapter arrivano al costruttore di Worker. Il lavoro è completato quando la configurazione di Worker può sovrascrivere entrambi gli adapter e gli utilizzi pertinenti rispettano coerentemente i valori iniettati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- ruby
- Ambito
- backend
- Tipo di issue
- Funzionalità
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100