Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: AsyncReader use BufReader #93

Closed
wants to merge 1 commit into from
Closed

Conversation

KKould
Copy link
Contributor

@KKould KKould commented Nov 6, 2024

before:
image

after:

+--------------------------------------------+---------+-------------+---------+----------+
|                                            | tonbo   | tonbo_foyer | rocksdb | tonbo_s3 |
+=========================================================================================+
| random range reads                         | 39207ms | 40021ms     | 36722ms | 55979ms  |
|--------------------------------------------+---------+-------------+---------+----------|
| random range reads                         | 38396ms | 39808ms     | 36585ms | 49915ms  |
|--------------------------------------------+---------+-------------+---------+----------|
| random range projection reads              | 28537ms | 29377ms     | 37077ms | 37680ms  |
|--------------------------------------------+---------+-------------+---------+----------|
| random range projection reads              | 28810ms | 29306ms     | 37003ms | 35845ms  |
|--------------------------------------------+---------+-------------+---------+----------|
| random range projection reads (4 threads)  | 17385ms | 18707ms     | 37233ms | 21270ms  |
|--------------------------------------------+---------+-------------+---------+----------|
| random range projection reads (8 threads)  | 17452ms | 17400ms     | 37396ms | 18609ms  |
|--------------------------------------------+---------+-------------+---------+----------|
| random range projection reads (16 threads) | 16728ms | 16596ms     | 35706ms | 17610ms  |
|--------------------------------------------+---------+-------------+---------+----------|
| random range projection reads (32 threads) | 16800ms | 16631ms     | 35659ms | 17723ms  |
+--------------------------------------------+---------+-------------+---------+----------+

@KKould
Copy link
Contributor Author

KKould commented Nov 6, 2024

How to calculate the io cost of minio to evaluate the difference

@KKould
Copy link
Contributor Author

KKould commented Nov 6, 2024

before:
5515cfd228ff977f41a7247dd30e2c3
e2e53c710f7055f118eae412c4e03f3

0.33G

after:
4045feed7e04cfe29a9a2eef9f2fd8b
fa6f744041b2321fae877b8b5a05123

1.11G

@KKould KKould closed this Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant