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

config: kipdb version up #208

Merged
merged 1 commit into from
Apr 23, 2024
Merged

Conversation

KKould
Copy link
Member

@KKould KKould commented Apr 23, 2024

What problem does this PR solve?

update kipdb version up

scan reduces consumption by 15%

FnckSQL: Full  Read by 'select * from t1'
                        time:   [107.80 ms 108.63 ms 109.44 ms]
                        change: [-16.754% -15.851% -14.928%] (p = 0.00 < 0.05)
                        Performance has improved.

SQLite: Full  Read by 'select * from t1'
                        time:   [44.343 ms 44.778 ms 45.563 ms]
                        change: [-2.2707% -0.4358% +1.5275%] (p = 0.67 > 0.05)
                        No change in performance detected.

FnckSQL: Point Read by 'select * from t1 where c1 = 1000'
                        time:   [30.191 µs 30.483 µs 30.813 µs]
                        change: [-0.0098% +1.3200% +2.7170%] (p = 0.10 > 0.05)
                        No change in performance detected.
Found 1 outliers among 10 measurements (10.00%)
  1 (10.00%) high mild

SQLite: Point Read by 'select * from t1 where c1 = 1000'
                        time:   [22.477 µs 22.875 µs 23.218 µs]
                        change: [-2.3707% -0.3802% +1.5986%] (p = 0.73 > 0.05)
                        No change in performance detected.
Found 1 outliers among 10 measurements (10.00%)
  1 (10.00%) high mild

FnckSQL: Range Read by 'select * from t1 where c1 > 500 and c1 < 1000'
                        time:   [402.87 µs 406.88 µs 411.86 µs]
                        change: [-15.453% -14.311% -13.144%] (p = 0.00 < 0.05)
                        Performance has improved.

SQLite: Range Read by 'select * from t1 where c1 > 500 and c1 < 1000'
                        time:   [135.34 µs 137.67 µs 140.19 µs]
                        change: [-1.5149% +0.5413% +2.7369%] (p = 0.63 > 0.05)
                        No change in performance detected.
Found 2 outliers among 10 measurements (20.00%)
  1 (10.00%) high mild
  1 (10.00%) high severe


Code changes

  • Has Rust code change
  • Has CI related scripts change

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Side effects

  • Performance regression: Consumes more CPU
  • Performance regression: Consumes more Memory
  • Breaking backward compatibility

Note for reviewer

@KKould KKould added the perf label Apr 23, 2024
@KKould KKould self-assigned this Apr 23, 2024
@KKould KKould merged commit af4b37b into KipData:main Apr 23, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant