ruby / ruby/json

Don't cache JRuby runtime in static fields

Aperta
#707 1 commento 0 reazioni 1 assegnatario Vedi su GitHub

@headius ci sta già lavorando.

Dal 21/11/2024.

jruby
Lingua principale
Ruby
Stelle
782
Fork
383
Merge medio
1g 18h
PR unite (30g)
10

Descrizione

There are a few places where we cache the JRuby runtime or runtime-specific objects in a static field, such as here with a WeakReference:

https://github.com/ruby/json/blob/7d2ad6d6556da03300a5aeadeeacaec563435773/java/src/json/ext/RuntimeInfo.java#L83-L91

The runtimes cached in weak references will not "leak" but it may take longer for the runtime to be collected than normal. In addition, the cost of traversing the weak reference may offset any gains from caching runtime objects.

These should be removed and cached in a runtime-safe way.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.