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

fix CI packaging #344

Merged
merged 1 commit into from
Oct 20, 2024
Merged

fix CI packaging #344

merged 1 commit into from
Oct 20, 2024

Conversation

var77
Copy link
Collaborator

@var77 var77 commented Oct 20, 2024

No description provided.

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

Hi! Looks like you've reached your API usage limit. You can increase it from your account settings page here: app.greptile.com/settings/usage

2 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

Copy link

Benchmarks

metric old new pct change
recall (after create) 0.956 0.955 -0.10%
recall (after insert) 0.000 0.000 -
select tps 23474.178 26395.381 +12.44%
select bulk(100) tps 34.429 32.597 -5.32%
select latency (ms) 0.715 ± 1.666𝜎 0.726 ± 1.396𝜎 +1.54%
select bulk(100) latency (ms) 890.200 ± 127.830𝜎 950.934 ± 156.539𝜎 +6.82%
create latency (ms) 400840.589 428054.566 +6.79%
insert tps 431.304 455.513 +5.61%
insert bulk(100) tps 4.625 4.841 +4.67%
insert latency (ms) 73.388 ± 15.791𝜎 69.726 ± 15.466𝜎 -4.99%
insert bulk(100) latency (ms) 6776.061 ± 145.904𝜎 6483.948 ± 129.306𝜎 -4.31%
disk usage (bytes) 8192008192.000 8192008192.000 -

@var77 var77 merged commit 924554a into main Oct 20, 2024
64 checks passed
@var77 var77 deleted the varik/fix-ci-package branch October 20, 2024 13:52
Copy link

codecov bot commented Oct 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

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