remove TODO

Author: sergejsvisockis

active-record/src/main/java/com/iluwatar/activerecord/base/RecordBase.java

@@ -16,14 +16,6 @@
 import lombok.RequiredArgsConstructor;
 
 /**
- * TODO: Think on potential simplification - perhaps for this use case we could just KISS by moving all the logic into the domain model itself.
- *  Hence at the same time each domain model will contain lots of boilerplate code.
- *  Let's think - Ruby has the record base parent object which encapsulates all the active record pattern logic.
- *  Therefore this one "the parent" hsa all the public API.
- *  This one adheres to the same pattern at the moment by implementing all the boilerplate persistence logic.
- *  Potentially two course of actions at this stage:
- *    1. Just KISS it - and move all the persistence logic into the domain models themselves and just copy-paste, copy-paste, copy-paste, copy-...
- *    2. Finish this abstract class and make things smoooth
  * An active record base supposed to hold all the necessary active record pattern logic.
  *
  * <p>This is the base class which is supposed to be extended by all the domain models that are
@@ -34,7 +26,8 @@
 @RequiredArgsConstructor
 public abstract class RecordBase<T extends RecordBase<?>> {
 
-  private static final String EXCEPTION_MESSAGE = "Couldn't execute database query for the following domain model :";
+  private static final String EXCEPTION_MESSAGE =
+      "Couldn't execute database query for the following domain model :";
 
   private static DataSource dataSource;