capybaralabs / d4j-store Goto Github PK

Postgres does not support the null byte in text-based fields, but some of the user generated strings sent by Discord may contain it.

Discord frequently changes / extends entities. Consider providing a migration strategy with Flyway or liquibase to deal with that.

Some of the calls in the DataUpdater to the repositories could be reduced to single bulk calls.

Some of the logic in the DataUpdater relies on fetching an entity, changing a property, then writing the entity back into the database.

This could be improved by more specialized, granular update methods that mutate the entity in the database directly.