Skip to content

Conversation

sdht0
Copy link
Contributor

@sdht0 sdht0 commented May 29, 2025

Currently, these warnings appear on the latest version of gcc.

@sdht0 sdht0 requested review from ray6080, mewim and andyfengHKU May 29, 2025 22:49
@sdht0 sdht0 changed the title chore: remove all code warnings, enforce in CI chore: remove all code warnings May 29, 2025
@sdht0 sdht0 marked this pull request as draft May 29, 2025 23:28
Copy link

codecov bot commented May 29, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.66%. Comparing base (48d4611) to head (2ea92b3).
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5471      +/-   ##
==========================================
- Coverage   86.67%   86.66%   -0.01%     
==========================================
  Files        1419     1419              
  Lines       62220    62218       -2     
  Branches     7636     7631       -5     
==========================================
- Hits        53930    53923       -7     
- Misses       8111     8115       +4     
- Partials      179      180       +1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link

github-actions bot commented May 29, 2025

Benchmark Result

Master commit hash: 412d632e0880ae212a1995717b4e66434e5edce9
Branch commit hash: e8e9dd17de2105519544a64f07743440343feb0b

Query Group Query Name Mean Time - Commit (ms) Mean Time - Master (ms) Diff
aggregation q24 664.48 678.68 -14.19 (-2.09%)
aggregation q28 6535.89 6510.43 25.46 (0.39%)
filter q14 59.61 60.91 -1.30 (-2.13%)
filter q15 63.17 61.79 1.37 (2.22%)
filter q16 274.13 276.09 -1.96 (-0.71%)
filter q17 380.13 380.40 -0.28 (-0.07%)
filter q18 1854.81 1866.10 -11.28 (-0.60%)
filter zonemap-node 22.42 22.41 0.01 (0.05%)
filter zonemap-node-lhs-cast 22.15 23.00 -0.85 (-3.72%)
filter zonemap-node-null 21.96 22.72 -0.76 (-3.36%)
filter zonemap-rel 5724.22 5819.87 -95.64 (-1.64%)
fixed_size_expr_evaluator q07 631.50 619.27 12.23 (1.97%)
fixed_size_expr_evaluator q08 908.23 909.85 -1.62 (-0.18%)
fixed_size_expr_evaluator q09 912.59 902.46 10.13 (1.12%)
fixed_size_expr_evaluator q10 198.60 191.04 7.56 (3.96%)
fixed_size_expr_evaluator q11 199.72 190.10 9.62 (5.06%)
fixed_size_expr_evaluator q12 175.41 167.44 7.97 (4.76%)
fixed_size_expr_evaluator q13 1530.81 1529.94 0.87 (0.06%)
fixed_size_seq_scan q23 56.40 44.74 11.66 (26.07%)
join q29 793.75 769.73 24.02 (3.12%)
join q30 1622.87 1570.87 52.00 (3.31%)
join q31 5.47 5.29 0.18 (3.42%)
join SelectiveTwoHopJoin 44.32 46.20 -1.88 (-4.07%)
ldbc_snb_ic q35 9.29 10.24 -0.95 (-9.30%)
ldbc_snb_ic q36 80.34 87.04 -6.70 (-7.70%)
ldbc_snb_is q32 2.48 3.09 -0.61 (-19.75%)
ldbc_snb_is q33 10.47 13.31 -2.84 (-21.35%)
ldbc_snb_is q34 1.27 1.24 0.04 (3.02%)
limit push-down-limit-into-distinct 2010.68 1947.34 63.34 (3.25%)
multi-rel multi-rel-large-scan 1794.63 1690.19 104.44 (6.18%)
multi-rel multi-rel-lookup 4.77 6.25 -1.48 (-23.65%)
multi-rel multi-rel-small-scan 185.30 205.09 -19.79 (-9.65%)
order_by q25 69.46 64.74 4.72 (7.29%)
order_by q26 380.35 402.11 -21.76 (-5.41%)
order_by q27 1308.95 1318.99 -10.03 (-0.76%)
recursive_join recursive-join-bidirection 378.67 365.51 13.15 (3.60%)
recursive_join recursive-join-dense 7063.37 7066.79 -3.42 (-0.05%)
recursive_join recursive-join-path 23227.94 23095.24 132.70 (0.57%)
recursive_join recursive-join-sparse 10.10 10.09 0.01 (0.13%)
recursive_join recursive-join-trail 6959.89 7062.81 -102.92 (-1.46%)
scan_after_filter q01 101.60 101.14 0.45 (0.45%)
scan_after_filter q02 89.30 89.18 0.13 (0.14%)
shortest_path_ldbc100 q37 77.56 73.13 4.43 (6.05%)
shortest_path_ldbc100 q38 331.18 304.53 26.65 (8.75%)
shortest_path_ldbc100 q39 90.49 68.88 21.61 (31.38%)
shortest_path_ldbc100 q40 505.16 504.71 0.44 (0.09%)
var_size_expr_evaluator q03 2044.84 2040.18 4.66 (0.23%)
var_size_expr_evaluator q04 2180.27 2414.28 -234.01 (-9.69%)
var_size_expr_evaluator q05 2630.04 2608.75 21.29 (0.82%)
var_size_expr_evaluator q06 1276.05 1285.92 -9.87 (-0.77%)
var_size_seq_scan q19 1347.30 1357.26 -9.96 (-0.73%)
var_size_seq_scan q20 2473.99 2653.19 -179.20 (-6.75%)
var_size_seq_scan q21 2202.27 2261.21 -58.93 (-2.61%)
var_size_seq_scan q22 108.62 109.70 -1.08 (-0.98%)

@sdht0 sdht0 marked this pull request as ready for review May 30, 2025 03:35
@sdht0 sdht0 merged commit 80cbaf6 into master May 30, 2025
53 of 54 checks passed
@sdht0 sdht0 deleted the code-warnings branch May 30, 2025 14:59
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.

3 participants