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

ROSS Message Size should be an override configuration #202

Open
nmcglo opened this issue Dec 9, 2019 · 0 comments
Open

ROSS Message Size should be an override configuration #202

nmcglo opened this issue Dec 9, 2019 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@nmcglo
Copy link
Member

nmcglo commented Dec 9, 2019

Requiring the ROSS message size to be included in the configuration file seems unnecessary to me. Way more often than not, these message sizes are defined at compile time, yet we require a runtime configuration value to specify the size. There's even a check that performs a sizeof() call and compares it with what is configured.

It might be a good idea to make that key:value pair in the configuration file an optional override for when it may be necessary.

@nmcglo nmcglo added the enhancement New feature or request label Dec 9, 2019
@nmcglo nmcglo self-assigned this Dec 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant