coinbase / coinbase/temporal-ruby

Injecting the metrics and log adapter into the Worker object

Aperta
#136 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.