Przeciek pamięci w Express.js z EventSource

Question

myślę używam do wycieku pamięci z ekspresowe aplikacji przy podłączeniu x liczba EventSource klientów do niego. Po połączeniu klientów i wysłaniu im x wiadomości i ich rozłączeniu, moja aplikacja zwolni tylko niewielką ilość przydzielonego Heap/RSS.

Aby to potwierdzić, zapisałem Heapdump podczas uruchamiania serwera i jeden po podłączeniu 7000 klientów do niego i wysyłania x wiadomości do każdego klienta. Czekałem chwilę, aby dać GC szansę na oczyszczenie się przed zrobieniem zdjęcia sterty.

Aby porównać migawki sterty, wczytałem je w widoku profilu narzędzi programistów Chrome i wybrałem tryb "Porównanie".

Moje pytania są następujące:

1) Jak interpretować te liczby? (Dla porównania patrz załączony sterty snapshot zrzut ekranu.)

2) Na przykład wygląda na to, że gniazdo obiektów nie prawie za darmo wszelkie obiekty w ogóle, czy to prawda?

3) Czy mogę podać mi więcej wskazówek, aby zbadać problem?

Answer 1

Możesz być wolny od wycieków pamięci i jako bonus uniknąć śmieciarza. Wszystko co musisz zrobić, to odpytywanie obiektów.

Można zrobić coś takiego

var clientsPool = new Array(1000); 
var clientsConnected = []; 

Gdy nowy klient łączy, robisz

var newClient = clientsPool.pop(); 
//set your props here 
clientsConnected.push(newClient); 

to niesamowity sposób na uniknięcie śmieciarza i uniemożliwić wyciek pamięci. Oczywiście, jest trochę więcej pracy i będziesz musiał to ostrożnie zarządzać, ale jest to warte swojej wydajności.

Jest niesamowita rozmowa o tym, tutaj idziesz https://www.youtube.com/watch?v=RWmzxyMf2cE

Answer 2

Jak na mój komentarz ...

Javascript nie można wyjaśnić sekcję pamięci powinno coś być skierowany na nią około 2 lat temu ktoś znalazł exploit i szybko został zamknięty tak i to działa tak

var someData = ["THIS IS SOME DATA SAY IT WAS THE SIZE OF A SMALL APPLICATION"]; 
var somePointer = someData[0]; 
delete someData; 

potem wstrzykuje aplikację do somePointer jak to było odniesienie do miejsca w pamięci, kiedy nie było żadnych danych teraz. hej, presto, wstrzyknęłaś pamięć.

Więc jeśli jest mowa jak wyżej somePointer = someData[0]; nie można zwolnić pamięć aż delete someData więc trzeba usunąć wszelkie odniesienia do cokolwiek chcesz czyszczących w przypadku ALL_CLIENTS.push(this); na linii 64 jest dokonanie Państwa pamięci systemowej dostępne za pośrednictwem ALL_CLIENTS, więc co można zrobić, to

Linia 157

_.each(ALL_CLIENTS, function(client, i) { 
        var u; // holds a undefined value (null, empty, nothing) 
        client.close(); 
        //delete ALL_CLIENTS[i]; 
        ALL_CLIENTS[i] = u; 
        ALL_CLIENTS.unused++; 
       }); 

Na innym uwaga to nie jest pamięć wycieku przeciek pamięci jest powiedzieć, że ten serwer go zamknąć, jeśli pamięć nie zwolnić po jej wyjściu wtedy masz przeciek pamięci, jeśli tak oczyść pamięć za sobą, to nie jest przeciek, to po prostu kiepskie zarządzanie pamięcią.

Dzięki @Magusowi za wskazanie, że usunięcie nie jest najlepszą rzeczą, jakiej możesz użyć, ale nigdy nie poleciłbym implementacji ograniczającej struktury, ale możesz spróbuj

Linia 27: ALL_CLIENTS.unused = 0;

Linia 64:

var u; 
if(ALL_CLIENTS.unused > 0){ 
    for(var i = 0; i < ALL_CLIENTS.length; i++){ 
     if(ALL_CLIENTS[i] == u){ 
      ALL_CLIENTS[i] = this; 
      ALL_CLIENTS.unused--; 
      i = ALL_CLIENTS.length; 
     } 
    } 
}else{ 
    ALL_CLIENTS.push(this); 
}