Defensive copies of payloads trigger G1 humongous allocations
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 45/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Tranquilla
- Stack tecnologico
- java
- Ambito
- performance
Direzione di ricerca
Inizia in Message.java esaminando il costruttore e le copie difensive di getPayload(), quindi ispeziona l’allocazione di decodifica a cui si fa riferimento intorno alla riga 98 e il ridimensionamento di BAOS intorno alla riga 132. Confronta il comportamento delle allocazioni per payload di grandi dimensioni e determina se il costruttore e il getter no-copy proposti possono preservare il comportamento esistente di Message; done deve includere prove della riduzione delle allocazioni humongous non necessarie.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Message constructor and getPayload() make defensive copies of large payloads, causing G1 humongous allocations
Describe the bug
Message makes two unconditional defensive copies of the payload byte array:
// Message.java — constructor
this.payload = requireNonNull(payload, "payload").clone();
// Message.java — getter
public byte[] getPayload() {
return payload.clone();
}
On a dev machine (8 vCPUs, 18 GB heap) for payloads larger than ~8 MB (half of a 16 MB G1 region), each clone is a G1 humongous allocation and may directly trigger a GC cycle. Humongous allocations degrade the overall system performance.
Possible Solution
Should Message class has no-copy constructor and payload getter using which will reduce unnecessary memory allocations?
Additional details
There are more allocations, e.g. array allocation during decoding or underlying BAOS array resizing. But these were not present on JFR recordings.
Library version used
aws-eventstream 1.0.1 (via aws-sdk 2.41.24)
JDK version used
21.0.10
Operating System and version
Linux
- Lingua principale
- Java
- Stelle
- 5
- Fork
- 6
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un 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 awslabs/aws-eventstream-java
-
New Version Release?Aperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
awslabs/aws-eventstream-java#3 · 1 reazione ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
awslabs/aws-eventstream-java#2 · 1 commento ·
Tutte le issue di awslabs/aws-eventstream-java
Issue simili
-
`Processing lsp` never exits and leaves orphaned processesForse già presa @overcast302 l’ha presa oggi. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
processing/processing4#1578 · 1 commento ·
-
ASM is not up-to-dateAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 60/100
I maintainer di solito rispondono entro 1 giorno
-
[BUG] S3 CORS responses omit Access-Control-Allow-Credentials for matched originsForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
floci-io/floci#5369 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
securityHeaders replaces a route's own Content-Security-Policy (0.9.9; weakens embedders' pages)Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
sqlcipher/sqlcipher-android#97 · 1 commento ·