Skip to content
Snippets Groups Projects

change bandwidth type from int to int64 to prevent overflow

Closed luciole requested to merge luciole/snowflake:40185-overflow-in-bandwidth into main

Closes #40185 (closed). Instead of using uint64 for the bandwidth unit (as I'd originally suggested), I used int64, because GetStat() in bytesNullLogger returns -1 for the inbound and outbound bandwidth values (otherwise we'd have to find a different way to indicate that bandwidth data wasn't logged, and int64 is already much more than enough for bandwidth logging).

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading