Add TODO for SegFault Factory in GetResultSet

include/caosdb/transaction.h

@@ -408,6 +408,10 @@ public:
       CAOSDB_LOG_ERROR(logger_name)
         << "GetResultSet was called before the transaction has terminated. This is a programming "
            "error of the code which uses the transaction.";
+      // TODO(tf) This is a really bad SegFault factory. When the transaction
+      // terminates and the result_set is being overriden, the unique_ptr
+      // created here will be deleted and any client of the return ResultSet
+      // will have a SegFault.
     } else if (this->GetStatus().GetCode() == StatusCode::SPOILED) {
       CAOSDB_LOG_ERROR(logger_name)
         << "GetResultSet was called on a \"spoiled\" transaction. That means "