Use SIGUSR1 to softstop #28

Lixfel · 2024-01-16T22:03:09+01:00

Lixfel hat

2024-01-16 22:03:09 +01:00

kommentiert

Besitzer

Wait for merged SpigotCore signal handling prior to merge.

Lixfel hat 1 Commit 2024-01-16 22:03:09 +01:00 hinzugefügt

Use SIGUSR1 to softstop

SteamWarCI Build successful

Details

ab5c965f20

YoyoNow hat 2024-01-16 22:19:25 +01:00 überprüft

src/de/steamwar/bungeecore/Subserver.java

						
				@ -143,0 +139,4 @@

				        try {

				            long pid = process.pid();

				            if(checkpoint)

				                pid = process.children().findAny().map(ProcessHandle::pid).orElse(pid);

YoyoNow hat

2024-01-16 22:19:25 +01:00

kommentiert

Besitzer

Ich glaube hier macht es mehr Sinn:

process.children().map(ProcessHandle::pid).findAny().orElse(pid);

Bzw warum genau machst du ein findAny? Würde nicht auch ein findFirst hier reichen?

Ich glaube hier macht es mehr Sinn: ``` process.children().map(ProcessHandle::pid).findAny().orElse(pid); ``` Bzw warum genau machst du ein findAny? Würde nicht auch ein findFirst hier reichen?

Lixfel hat

2024-01-16 22:25:30 +01:00

kommentiert

Autor

Besitzer

Ist findAny hier nicht das gleiche wie findFirst? Es wird sowieso nur 1 Kind geben (und das eigentlich immer).

YoyoNow hat

2024-01-16 22:27:10 +01:00

kommentiert

Besitzer

Achso, ja dann ist es egal. Und das hängt davon ab ob der Stream ein parallel Stream ist oder nicht. Lass es dann einfach so.

YoyoNow markierte diese Unterhaltung als gelöst