[GitHub] [incubator-kvrocks] git-hulk commented on pull request #956: Move TCL test unit/introspection to Go case

[GitBox <gi...@apache.org>]

git-hulk commented on PR #956:
URL: https://github.com/apache/incubator-kvrocks/pull/956#issuecomment-1271076030

   @tisonkun  The test `DEBUG will freeze server` also failed casually in Go, because it may use a different connection between `Debug Sleep 2.2` and `Set a b` command and `Set a b` won't wait for the debug command when it's served by the different worker.
   
   Below is TCP packets, the debug sleep command is sent by connection(port 49225) and the set command is in connection(port 49221):
   ```
   11:24:50.845605 IP localhost.49225 > localhost.16379: Flags [P.], seq 1:36, ack 1, win 6379, options [nop,nop,TS val 1201924138 ecr 643167600], length 35
   ....E..W..@.@............I?./..C.L.......K.....
   G..*&U.p*3
   $5
   DEBUG
   $5
   sleep
   $3
   2.2
   
   11:24:50.845681 IP localhost.49221 > localhost.16379: Flags [P.], seq 97:124, ack 301, win 6375, options [nop,nop,TS val 3933309289 ecr 2680271241], length 27
   ....E..O..@.@............E?. .F..........C.....
   .q.i....*3
   $3
   set
   $1
   a
   $1
   b
   ```
   
   May be we can limit the pool size to 1 to workaround this issue.
   


-- 
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: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org