Fix soft shutdown e limita jogadores #196
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…rrou. Ele vai ser usado para que MiniTrucoServer mantenha uma coleção das threads de JogadorConectado abertas (pra corrigir um bug e saber quantos usuários estão conectados)
…reads), esperando as threads de JogadorConectado (porque a JVM não espera threads virtuais)
|
Parte de #173 |
chesterbr
commented
Aug 5, 2023
| @@ -90,7 +114,28 @@ public static void aceitaConexoes() { | |||
| ServerLogger.evento(e, "Erro de I/O ao fechar ServerSocket"); | |||
| } | |||
| } | |||
| ServerLogger.evento("Servidor não está mais escutando; aguardando finalização dos jogadores conectados."); | |||
| ServerLogger.evento("Servidor não está mais escutando (porta liberada)"); | |||
| aguardaThreadsJogadoresFinalizarem(); | |||
There was a problem hiding this comment.
FWIW, não, eu não posso dar join delas todas. Aparentemente isso não funciona tão bem com Virtual Threads; vou investigar mais a fundo futuramente.
There was a problem hiding this comment.
(talvez não tenha funcionado porque eu tentei fazer isso naquele handler do signal de USR1?)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
A introdução de virtual threads (#195) causou um bug interessante: como as threads de
JogadorConectadonão são threads propriamente ditas, a VM não espera elas encerrarem pra dar shutdown, fazendo com que o soft shutdown (#185) falhe (o servidor cai imeditamente ao finalizar a thread que escuta conexões).Para corrigir isso, foi preciso manter uma collection com as threads de
JogadorConectadoem execução (o que, por sua vez, exigiu que fosse implementado um callback naquela classe para queMiniTrucoServersaiba quando um jogador finaliza e tire ele da collection). Quando o listen é interrompido, a thread que escuta as conexões (a única coisa mantendo a JVM no ar nessa hora) espera que não haja mais nenhum jogador conectado antes de encerrar.Com essa coleção em mãos, ficou fácil implementar dois outros itens de #38: um jeito fácil de ver quantos jogadores estão conectados (bastou logar um evento quando a coleção é modificada: "Jogadores conectados: <tamanho da coleção>", que é facilmente
grepável) e um limite do número de jogadores conectados (quando uma conexão entra e a coleção já está no limite daquele tamanho, manda uma notificação toast e desconecta).Poderia fazer o máximo ser alterável dinamicamente, mas vamos pensar MVP aqui.