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 platform issues with md5sum utility #4077

Merged
merged 5 commits into from
Aug 14, 2024
Merged

Fix platform issues with md5sum utility #4077

merged 5 commits into from
Aug 14, 2024

Conversation

mxwli
Copy link
Contributor

@mxwli mxwli commented Aug 14, 2024

Description

Please include a summary of the changes and the related issue (if applicable). Please also include
relevant motivation and context.

Fixes # (issue)

Contributor agreement

@mxwli mxwli merged commit ed6d847 into master Aug 14, 2024
16 checks passed
@mxwli mxwli deleted the m5platformfix branch August 14, 2024 16:55
Copy link

codecov bot commented Aug 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.13%. Comparing base (c76e05c) to head (8a02d35).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4077   +/-   ##
=======================================
  Coverage   84.13%   84.13%           
=======================================
  Files        1301     1301           
  Lines       51206    51206           
  Branches     7119     7120    +1     
=======================================
+ Hits        43080    43084    +4     
+ Misses       7983     7978    -5     
- Partials      143      144    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

Benchmark Result

Master commit hash: c76e05c400119276d4a65e727ebcb1341f6e26b4
Branch commit hash: b5f5f9e886e83234d99c9d06d20a1f2a481ca866

Query Group Query Name Mean Time - Commit (ms) Mean Time - Master (ms) Diff
aggregation q24 679.16 676.41 2.75 (0.41%)
aggregation q28 10892.72 11022.64 -129.92 (-1.18%)
filter q14 152.42 158.37 -5.95 (-3.76%)
filter q15 152.73 151.78 0.95 (0.63%)
filter q16 332.34 335.19 -2.85 (-0.85%)
filter q17 472.33 475.15 -2.82 (-0.59%)
filter q18 1952.29 1978.89 -26.60 (-1.34%)
fixed_size_expr_evaluator q07 566.68 568.32 -1.64 (-0.29%)
fixed_size_expr_evaluator q08 776.23 781.76 -5.53 (-0.71%)
fixed_size_expr_evaluator q09 776.91 782.37 -5.45 (-0.70%)
fixed_size_expr_evaluator q10 264.31 264.78 -0.47 (-0.18%)
fixed_size_expr_evaluator q11 258.53 259.20 -0.67 (-0.26%)
fixed_size_expr_evaluator q12 256.94 257.61 -0.67 (-0.26%)
fixed_size_expr_evaluator q13 1493.74 1493.69 0.05 (0.00%)
fixed_size_seq_scan q23 145.72 144.46 1.27 (0.88%)
join q31 11.80 12.45 -0.65 (-5.26%)
ldbc_snb_ic q35 854.08 807.41 46.67 (5.78%)
ldbc_snb_ic q36 58.06 30.59 27.46 (89.78%)
ldbc_snb_is q32 10.23 9.58 0.65 (6.79%)
ldbc_snb_is q33 17.95 18.63 -0.68 (-3.66%)
ldbc_snb_is q34 8.85 8.52 0.33 (3.89%)
multi-rel multi-rel-large-scan 2975.53 2939.43 36.11 (1.23%)
multi-rel multi-rel-lookup 51.47 59.51 -8.04 (-13.51%)
multi-rel multi-rel-small-scan 54.02 57.52 -3.50 (-6.08%)
order_by q25 158.41 154.81 3.60 (2.32%)
order_by q26 473.51 470.33 3.17 (0.67%)
order_by q27 1431.52 1439.31 -7.80 (-0.54%)
scan_after_filter q01 196.13 198.27 -2.13 (-1.08%)
scan_after_filter q02 186.05 186.62 -0.57 (-0.30%)
shortest_path_ldbc100 q39 103.37 103.22 0.15 (0.15%)
var_size_expr_evaluator q03 2084.65 2090.18 -5.53 (-0.26%)
var_size_expr_evaluator q04 2276.00 2290.00 -14.00 (-0.61%)
var_size_expr_evaluator q05 2627.55 2617.13 10.42 (0.40%)
var_size_expr_evaluator q06 1353.71 1351.61 2.10 (0.16%)
var_size_seq_scan q19 1496.66 1489.71 6.95 (0.47%)
var_size_seq_scan q20 3185.98 3213.14 -27.16 (-0.85%)
var_size_seq_scan q21 2450.52 2454.96 -4.44 (-0.18%)
var_size_seq_scan q22 135.26 136.93 -1.67 (-1.22%)

ted-wq-x pushed a commit to ted-wq-x/kuzu that referenced this pull request Nov 14, 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.

2 participants