AlexZhao

Results 13 comments of AlexZhao

Hi @gaoran10 Is there any unassigned issue which I would like to pick up.

@abbccdda @hachikuji Comments have been addressed, could you continue to review the pr.

Not sure if it has something to do with this pr: ISSUE 3220-Autorecovery does not process underreplicated empty ledgers https://github.com/apache/bookkeeper/pull/3239 If network packet loss causes a bookie to be determined...

> java.lang.NullPointerException Yes. it is a bookkeeper client issue, and the bkclient built on the master branch. Sorry for submitting to the wrong place.

I am not sure if this issue is related to [#3321](https://github.com/apache/bookkeeper/pull/3321), which try to fix ledger handle leak issues @hangc0276 @codelipenghui

The error log context is as followed. Not sure if it can help. @zymap > 11338 Oct 27 18:02:42 test_host pulsar[166925]: 2022-10-27T18:02:42,055+0800 [BookKeeperClientWorker-OrderedExecutor-22-0] WARN org.apache.bookkeeper.client.RackawareEn 11338 semblePlacementPolicyImpl - Failed to...

> Does this error still happen? I am not sure about it is related to [#3321](https://github.com/apache/bookkeeper/pull/3321). In 3321, the recycling happened at the beginning of add entry operation. If it...

@stmatengss Hi, I'd like to try sanitizer detection support for C++ if no one else is working on it

> > Thanks for the great work! Looks like the design is pretty similar to the existing KV connector API. Why don't we just directly use the existing connector API?...

> [@zhaohaidao](https://github.com/zhaohaidao) you can aggregate connectors using the `MultiKVConnector`: https://github.com/vllm-project/vllm/blob/main/vllm/distributed/kv_transfer/kv_connector/v1/multi_connector.py @njhill Thank you very much, let me take a look