hailin0 commented on PR #2644:
URL: 
https://github.com/apache/incubator-seatunnel/pull/2644#issuecomment-1236554960

   > > please help to review @TyrantLucifer
   > 
   > After reading the jedis-related source codes, the call stack of the 
connection method is as follows:
   > 
   > jedis#set -> connection#executeCommand -> connection#sendCommand -> 
connection#connect
   > 
   > So I think add these codes is useless.
   > 
   > Redis e2e test cases has no problem in my local environment and github 
action environment, it's may be a problem with your local environment 
configuration.
   
   
   This depends on the performance of the operating environment.
   
   
   This function waiting for redis server start completed and connected to 
redis server successful. This requires calling Jedis#connect to guarantee.
   ```java
           given().ignoreExceptions()
                   .await()
                   .atMost(180, TimeUnit.SECONDS)
                   .untilAsserted(this::initJedis);
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to