Minor code clean up

• Change from

  return await self._client.mget(keys[0], *keys[1:])
  

  to the

  return await self._client.mget(*keys)
  

---

• Change from

  if isinstance(value, int) or value.isdigit():
     return int(value)
  

  to the

  if isinstance(value, int)
      return value
  if value.isdigit():
      return int(value)
  

---

• Refactor magic hidden converting under the hood from microseconds to seconds based only on variable type:

           if isinstance(expire, float):
               pexpire = int(expire * 1000)
               expire = None

• This seems to be unnecessary because on __del__ call self._client will be deleted too:

    def close(self):
       del self._client
       self._client = None
       self.__is_init = False
  
   __del__ = close
  

---

• Using class none: pass seems to be redundant because None is pickled and unpickled back. I do not understand the purpose of the

        value = pickle.loads(value, fix_imports=False, encoding="bytes")
  
        if value is none:
            return None
  

  because you can remove at all class none: pass class, and just do return value.

[Krukov]

Yep, thank to point it. I hope that is no doubts about socket_keepalive - the same setting we have in our browsers in it set to true. To be honest there is no reason why socket_timeout is exactly 0.1 by default. I just want to set it with a small amount based on my experience. We had an issues with cache when it stacked and we got the whole app down by the end just because timeout had a big value or didn’t set. I don’t think that we can count the value for socket timeout, but for cache it should be extremely small just because of the purpose of cache.

I saw some redis benchmarks and they said that average response time is 200 us ( 0.0002 sec) ! and 100 percentile took less then 3 ms ( 0.003 sec) (https://redis.io/topics/benchmarks) !

Do you have some suggestions for that value?

[Krukov]

@AIGeneratedUsername Can I ask you for review this PR ?