Wenn wir damit anfangen, Exceptions in diese Tabelle zu packen, haben wir dasselbe wie die Exception Tabelle.
Und die "normalen" Logs abzuschaffen sehe ich als absolut kontraproduktiv an, da…
Trotzdem müssen wir dem Spieler nicht etwas sagen, ohne das wir einen Namen zum Bannen im gegenzug bekommen
Nope, sagt dem (gebannten) Spieler etwas aus der DB, was er evtl. gar nicht wissen soll.
Das hier nach jedem create ein Get ausgeführt wird ist nicht nötig. Nirgends wird nach dem Create das Teil nochmal benutzt.
Hier sind überall sehr viele redundante Parameter in der Funktion, welche auch (fast) immer mit dem gleichen Funktionsauruf befüllt werden.
Da es sich hier um eine Write-Only Tabelle handelt, und wir später nicht nochmal die Spalte einzeln Querryn bzw. Löschen oder so, braucht es hier eigentlich keine Id und Indexe.
Warum muss Join und Leave doppelt gespeichert werden? s. SessionManager.java im Velocity
Wofür brauch es nochmal ein Slash vor dem Command, durch den Typen ist bekannt, das es sich um einen Command handelt