[michal__q]

Gdybyście się zastanawiali dlaczego wykop działa tak słabo.

Zastanówmy się jak można przekazać komunikat do aplikacji, że chcę zaplusować jakiś wpis, wystarczy wysłać komunikat Chcę zaplusować komentarz o id x
Więc mamy komendę + identyfikator, możemy to uzyskać robiąc taki request:

POST /articles/:id/votes

Nie potrzebujemy nic więcej, (oprócz jakiejś autoryzacji czy innych tokenów) w responsie też nie zwracajmy contentu, zwróćmy HTTP 204 No content
Tysiące ludzi lajkuje gównowpisy, zaoszczędzimy troche transferu. Chociaż, jeśli chcemy być bardziej dynamiczni to możemy zwrócić ilość lajków z jakiejś szybkiej bazy np redis gdzie są po prostu incrementowane. Na przykład tak: {"votes": int}
Analogiczne można zrobić z usuwaniem

DELETE /aticles/:id/votes

To samo w zrwotce 204 No content. Lub analogicznie jak wyżej {"votes": int}.

Jak to się robi na wykopie?
Sam endpoint nie jest jakoś straszny

/ajax2/wpis/voteUp/…

Ale ciekawe jest to co dostajemy w odpowiedzi, wydawać by się mogło, że wystarczy informacja czy sukces, czy nie?

for(;;);{"operations":[{"type":"content","html":"+: kiera1<\/a>, invictuss<\/a> +3627 innych<\/a><\/div>","find":"div.votersContainer:first","closest":"li"},{"type":"replace","html":"\n\t\t **+3629<\/span><\/b>\n\t\t \t*<\/i><\/a>\n\t\t *<\/i>\n\t\t\t<\/p>","closest":"p"}]}

Dostajemy JavaScript, który zawiera Jsona, który zawiera listę operacji (dokładnie to dwie), które zawierają HTML.
Są to operacje typu content, oznacza mniej więcej "wstaw zawartość pola html tam gdzie wskazuje pole find", oraz replace, które działa w zasadzie tak samo.

Warto zwrócić uwagę na dolny pasek

+: kiera1, invictuss +3859 innych

właściwie +x innych to licznik pomniejszony o dwie nazwy użytkowników z przodu. Nazwy użytkowników na początku czasami się zmieniają po akcji plusowania, ale czy muszą? Czy warto marnować zasoby na taką pierdołę?
Programiści wykopu stwierdzili, że muszą, dlatego cały portal działa jak gówno.
Nie dlatego, że jest dużo użytkowników, tylko dlatego, że jest słabo zaprojektowany.

#webdev #programowanie #restapi #maciejpsuje #wykop #mikroblog****

[olelek]

tylko dlatego, że jest słabo zaprojektowany.

@michal__q: I taki właśnie ma być ( ͡° ͜ʖ ͡°)

[Marmite]

@michal__q: kiedyś pytałem Macieja o to, powiedział że ta decyzja została podjęta z jakichś tam powodów, nawet brzmiały wtedy sensownie. Tylko chyba nie przemyśleli że to się będzie słabo skalować ( ͡° ͜ʖ ͡°) to było z 5-6 lat temu, od tamtej pory ilość informacji przepływająca przez Wykop pewnie mocno się zwielokrotniła

[oslet]

@michal__q ręce opadają. Development na poziomie studenta 1 roku

[oslet]

@michal__q a na backendzie pewnie jedno zapytanie pyta o ilość plusów, w kodzie robione jest i+1 i drugie zapytanie update zapisuje do bazy wynik takiego dodawania.. Oczywiście bez transakcji itd

[sokytsinolop]

@michal__q: #!$%@?, nie dlatego wykop tak wolno działa. Problem leży głębiej.

[michal__q]

@sokytsinolop: oczywiście, że leży głębiej, to tylko przedsmak zgrzybiałej architektury tego serwisu :)

[sokytsinolop]

@michal__q: niby oczywiste, ale to co napisałeś absolutnie nie musi mieć wpływu na szybkość działania

[michal__q]

@sokytsinolop: Tak, nie musi mieć. Po drodze może być jakiś cache, mogą być porobione jakieś optymalizacje, ale jak na to patrzę to jednak odnoszę wrażenie, że cała aplikacja tak wygląda.

[sokytsinolop]

@michal__q: tak się kiedyś po prostu robiło apki, bo nie było reacta i vue, ani API z którego korzystał cały front i tak bylo prościej. To się tak wydaje, że to nieoptymalne bo to brzydko wygląda, ale założę się ze react żeby wyrenderować nowy widok wykona znacznie więcej operacji.

[michal__q]

@sokytsinolop: Kiedyś może i tak, chociaż i tak się nie zgodzę do końca. Masz do aktualizacji ilość plusów w elemencie, czy na pewno potrzebujesz przesyłać cały html do tego? Przecież poczciwe jquery udostępnia ci możliwość łatwej podmiany zawartości elementu. Z resztą słowo klucz: kiedyś, czyli kiedy? 5 lat temu?

[t.....k]

@michal__q @oslet @sokytsinlop zazwyczaj to nie jest wina developerów a człowieka który nimi kieruje, być może PM nie rozumiał co chce osiągnąć, być może było to faktycznie pisane na kolanie, wgle krytykujecie jak byście sami nigdy nie pracowali w polskiej firmie ( ͡° ͜ʖ ͡°)

[sokytsinolop]

@michal__q: ja wiem, że można i powinno się to robić inaczej ale to nie to co wypisałeś spowalnia wykop.
@tlaziuk: ja nic nie krytykuję, zwracam tylko uwagę że to napisał op nie ma wpływu na szybkość działania serwisu

[object]

@michal__q: też tego nie rozumiem.
Co więcej jakiś czas temu widziałem informacje, że wykop jest przepisywany na coś nowszego / lepiej zrobionego.
Można zastanowić się ile czasu im to zajmie.. i według mnie to już dawno powinno być zrobione ( ͡° ͜ʖ ͡°)

[stawek]

założę się ze react żeby wyrenderować nowy widok wykona znacznie więcej operacji.

@sokytsinolop: i pewnie wcale nie będziesz w błędzie - front reddita też przepisali na react i działa toto jak gówno ¯\_(ツ)_/¯ i to raczej nie będzie kwestia jakości programistów bo co jak co ale reddit chyba moze sobie pozwolić na zatrudnienie tych z górnej półki...

[croppz]

front reddita też przepisali

@stawek: na szczęście zostawili też stary ( ͡° ͜ʖ ͡°)

[V.....n]

@sokytsinolop: co ma front do backendu? To zapytanie często idzie paręnaście sekund i to jest problemem.

[sokytsinolop]

@Verijon: no to tłumaczyłem to właśnie opowi