before process assertNoSeqNumberConflict, the tranlog has already added。i think the assert is not necessary。
No assertions are necessary: they are disabled in production builds after all. That doesn't mean they aren't useful. Even an assertion which can never fail is still useful as a piece of documentation, spelling out the assumptions that the code is making, and will block any changes that violate those assumptions in future.