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.

Sign up for GitHub

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

Jump to bottom

Remove Extra Platform Info That Is Not Used #475

Merged

waahm7 merged 2 commits into main from new-instance-types

Dec 12, 2024

Contributor

waahm7 commented Dec 12, 2024

Description of changes:

Remove cpu_group and numa_nodes information for now. These were added for a future optimization where we would auto-configure the best possible settings based on NUMA nodes, but we are not using this functionality yet. As a wise man once said

Remove this until needed.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

waahm7 added 2 commits

December 11, 2024 15:54


          remove uneeded stuff

2a93e37


          fix sample

d09dab6

graebm approved these changes

View reviewed changes

Contributor

graebm left a comment

it's "Red code is best code", but sure

waahm7 enabled auto-merge (squash)

December 12, 2024 00:16

waahm7 merged commit 58a7adc into main

33 of 34 checks passed

waahm7 deleted the new-instance-types branch

December 12, 2024 00:27

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet