hulk

Results 524 comments of hulk

@DenizPiri Thanks for your feedback. The WATCH has already in [our 2023 roadmap](https://github.com/orgs/apache/projects/199) and will start a discussion if someone has time to drive this feature. And for test cases,...

> 组织:科大讯飞 ADX 地点:中国安徽 联系方式:[[email protected]](mailto:[email protected]) 使用场景:程序化广告投放中的 uv 级素材缓存 ![微信图片_20230216165849](https://user-images.githubusercontent.com/23516980/219317086-41720eb0-d2da-4759-aba5-2707012fcec8.png) ![logo-1](https://user-images.githubusercontent.com/23516980/219317311-b9941372-e560-44fd-8cc3-d854b555af26.png) Thanks @HarmonYoung, will update to users soon.

> @git-hulk Thanks a lot for the reply! > > I was referring to tests that load DB files from older versions of KVrocks and make sure that it can...

Hi, @Eshcar @ShooterIT I have an idea to workaround this bug. We can add a new function to create a shared WriteBatch for the Multi-Exec command like the below: ```C++...

Yes, RocksDB also supports the `WriteBatchWithIndex` to implement the Read-Your-Own-Write feature, but it needs to involve all read operations, so I didn't plan to do it in the current stage....

@ShooterIT Do you mean `WriteBatchWithIndex` can only be used in transaction mode? I see the [MongoDB](https://github.com/mongodb/mongo/blob/a31cc114a89a3645e97645805ba77db32c433dce/src/mongo/db/storage/rocks/rocks_recovery_unit.cpp#L245-L260) will search the `txn_write_batch` first if exists then from the DB, so it should...

oh...I got your point now. What I mean is to implement the atomic write operations first, then involves read operations at the next stage.

@ShooterIT Yes, we cannot require users to avoid this, what I mean is to separate them into two PRs to make the code review simple.

Hi @FZambia `go-redis` has compatible with this error message for Kvrocks, can see [error.go#L23](https://github.com/go-redis/redis/blob/4bda6ec2fb4a836bbffc0032ad3a0f2f79528ccb/error.go#L23). So you can upgrade your library to https://github.com/go-redis/redis/releases/tag/v9.0.0-rc.2, or copy out the `Run` and `HasErrorPrefix` functions...

@FZambia Yes, we will also fix this issue on Kvrock's side soon.