Yong Zhang

Results 50 comments of Yong Zhang

Thanks @dlg99 , I will take a look

Remove it to the next release because I think it needs more people to confirm.

@dlg99 Thanks for your information. But looks like the maxAddsInProgressLimit only applied on the v3 request?

I tried the backpressure with V2 protocol. It doesn't work. Because it is a server limitation, when there has a server slow to respond, the other server will respond successfully,...

The test output looks like works as expected, I will try to add more tests and clean the code. logs ``` 2022-10-11T16:56:19,449 - INFO - [BookKeeperClientWorker-OrderedExecutor-0-0:BookieClientMemoryCounterTest$1@68] - Write state changed...

@RaulGracia I saw the logs in step 2: ``` 2022-07-20T14:45:47.426475848+00:00 stdout F 2022-07-20 14:45:47,426 7065710 [ZKC-connect-executor-0-SendThread(zookeeper-client:2181)] DEBUG org.apache.zookeeper.ClientCnxn - Reading reply session id: 0x2016bc13e0a001c, packet:: clientPath:/pravega/pravega/bookkeeper/ledgers/00/0000/L3246 serverPath:/pravega/pravega/bookkeeper/ledgers/00/0000/L3246 finished:false header:: 146,4...

@lordcheng10 I think you need to add more details to the proposal to know how this work. Like when will the migration does, how it does migration, and what will...

@hangc0276 Could you please take a look as well? Because it looks like is an improvement of the skip unrecoverable ledger.