Skip to content

Fix file sizes#64

Merged
JackPGreen merged 1 commit into
hazelcast:masterfrom
JackPGreen:fix-file-sizes
May 18, 2026
Merged

Fix file sizes#64
JackPGreen merged 1 commit into
hazelcast:masterfrom
JackPGreen:fix-file-sizes

Conversation

@JackPGreen

Copy link
Copy Markdown
Contributor

In #56, it was discovered the pipeline was using wrong filesizes (MiB vs MB).

Although that updated future files, that left older files stuck.

I've gone through and updated to be consistent, based on logic like:

curl -sI https://repository.hazelcast.com/download/management-center/hazelcast-management-center-5.11.0.zip | awk '/^[Cc]ontent-[Ll]ength:/ {print $2}' | tr -d '\r'

printf '%s\n' 326254801 | awk '{ printf "%.1f MB\n", $1 / 1000000 }'

For platform, I've used whole-sizes, for MC, 1 decimal place to be consistent with existing format.
I've left the rest as parsing disparate formats is too complex for a throwaway activity.

In hazelcast#56, it was discovered the pipeline was using wrong filesizes (`MiB` vs `MB`).

Although that updated _future_ files, that left older files stuck.

I've gone through and updated to be consistent, based on logic _like_:
```
curl -sI https://repository.hazelcast.com/download/management-center/hazelcast-management-center-5.11.0.zip | awk '/^[Cc]ontent-[Ll]ength:/ {print $2}' | tr -d '\r'

printf '%s\n' 326254801 | awk '{ printf "%.1f MB\n", $1 / 1000000 }'
```

For platform, I've used whole-sizes, for MC, 1 decimal place to be consistent with existing format.

@nishaatr nishaatr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It will interesting to see if website actually takes these new values.
Nonetheless, best to fix and make it consistent

@JackPGreen JackPGreen merged commit a6ff50b into hazelcast:master May 18, 2026
3 of 4 checks passed
@JackPGreen JackPGreen deleted the fix-file-sizes branch June 23, 2026 12:01
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