OpenAtomFoundation / pikiwidb

a high-performance, large-capacity, multi-tenant, data-persistent, strong data consistency based on raft, Redis-compatible elastic KV data storage system based on RocksDB
BSD 3-Clause "New" or "Revised" License
202 stars 63 forks source link

bug: Conn has unread data #173

Closed 578223592 closed 8 months ago

578223592 commented 8 months ago

Is this a regression?

Yes

Description

although https://github.com/OpenAtomFoundation/pikiwidb/pull/169 has fix some Conn has unread data bug ,but there is still 5 Conn has unread data。 some link: first report : https://github.com/OpenAtomFoundation/pikiwidb/pull/150 https://github.com/OpenAtomFoundation/pikiwidb/pull/169 https://github.com/OpenAtomFoundation/pikiwidb/issues/167

Please provide a link to a minimal reproduction of the bug

git checkout unstable
cd tests
go test

Screenshots or videos

Running Suite: Tests Suite - /home/swx/projects/pika_dev/pikiwidb_backup/tests
==============================================================================
Random Seed: 1708686795

Will run 33 of 33 specs
••••••••••••••••••••••redis: 2024/02/23 19:14:29 pool.go:352: Conn has unread data
redis: 2024/02/23 19:14:29 pool.go:352: Conn has unread data
redis: 2024/02/23 19:14:29 pool.go:352: Conn has unread data
•redis: 2024/02/23 19:14:30 pool.go:352: Conn has unread data
••••redis: 2024/02/23 19:14:41 pool.go:352: Conn has unread data
••••••

Ran 33 of 33 Specs in 95.703 seconds
SUCCESS! -- 33 Passed | 0 Failed | 0 Pending | 0 Skipped
PASS
ok      github.com/OpenAtomFoundation/pikiwidb/tests    95.708s

Please provide the version you discovered this bug in (check about page for version information)

https://github.com/OpenAtomFoundation/pikiwidb/commit/5ea7bc0949cbd49122633cf086322143121c5985

Anything else?

No response

578223592 commented 8 months ago

目前版本测试没复现了