fix: review comments

Author: vyavdoshenko

src/server/hset_family.cc

@@ -1164,7 +1164,6 @@ void HSetFamily::HRandField(CmdArgList args, const CommandContext& cmd_cntx) {
       }
 
       if (string_map->Empty()) {  // Can happen if we use a TTL on hash members.
-        // Use type-safe deletion (fixes #5316)
         auto res_it = db_slice.FindMutable(db_context, key, OBJ_HASH);
         if (res_it) {
           res_it->post_updater.Run();

src/server/set_family.cc

@@ -481,7 +481,6 @@ OpResult<uint32_t> OpAdd(const OpArgs& op_args, std::string_view key, const NewE
   // to overwrite the key. However, if the set is empty it means we should delete the
   // key if it exists.
   if (overwrite && (vals_it.begin() == vals_it.end())) {
-    // Use type-safe deletion with OBJ_SET (fixes #5316)
     auto res_it = db_slice.FindMutable(op_args.db_cntx, key, OBJ_SET);
     if (res_it) {
       res_it->post_updater.Run();

src/server/string_family.cc

@@ -285,7 +285,6 @@ OpResult<int64_t> OpIncrBy(const OpArgs& op_args, string_view key, int64_t incr,
   auto& db_slice = op_args.GetDbSlice();
 
   // we avoid using AddOrFind because of skip_on_missing option for memcache.
-  // Use type-safe FindMutable with OBJ_STRING (fixes #5316)
   auto res = db_slice.FindMutable(op_args.db_cntx, key, OBJ_STRING);
 
   if (!res) {
@@ -384,7 +383,6 @@ OpResult<array<int64_t, 5>> OpThrottle(const OpArgs& op_args, const string_view
   // Cost of this request
   const int64_t increment_ns = emission_interval_ns * quantity;  // should be nonnegative
 
-  // Use type-safe FindMutable with OBJ_STRING (fixes #5316)
   auto res = db_slice.FindMutable(op_args.db_cntx, key, OBJ_STRING);
   const int64_t now_ns = GetCurrentTimeNs();