Fix various player save issues (performance penalty on sql backends + bugs)
authorLoïc Blot <loic.blot@unix-experience.fr>
Fri, 4 Jan 2019 09:20:04 +0000 (10:20 +0100)
committerLoïc Blot <loic.blot@unix-experience.fr>
Fri, 4 Jan 2019 09:20:04 +0000 (10:20 +0100)
commitc1d7dbfc38e8f48b42d957afad2696a8e56b740b
tree53db583fc199c9eb096c15e6ffa6c85539a54c0d
parent07177190734b0e5c17be18d0bf5d509c5226af2f
Fix various player save issues (performance penalty on sql backends + bugs)

* PostgreSQL & SQLite3 doesn't setModified(false) on RemotePlayer, then player is saved on each server save call. This results in heavy useless writes.
* PostgreSQL & SQLite3 ack engine meta write whereas db commit hasn't been performed. If commit failed write has failed. We mustn't notify engine write is done.
* serializing player meta must not setModified(false) because it didn't ensure write has been done
* add RemotePlayer::on_successfull_save callback to do the flag update on a successful save
src/database/database-files.cpp
src/database/database-postgresql.cpp
src/database/database-sqlite3.cpp
src/remoteplayer.cpp
src/remoteplayer.h