fix: heap configuration from zowe.yaml

[achmelo]

Description

Allow users to configure JVM heap parameters from standard zowe.yaml configuration file.

Linked to #2559
Part of the # (epic)

Type of change

Please delete options that are not relevant.

• (fix) Bug fix (non-breaking change which fixes an issue)
• (feat) New feature (non-breaking change which adds functionality)
• (docs) Change in a documentation
• (refactor) Refactor the code
• (chore) Chore, repository cleanup, updates the dependencies.
• (BREAKING CHANGE or !) Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas. In JS I did provide JSDoc
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• The java tests in the area I was working on leverage @nested annotations
• Any dependent changes have been merged and published in downstream modules

For more details about how should the code look like read the Contributing guideline

[balhar-jakub]

Looks good

[JirkaAichler]

Do we want to have the same heap size for all components? Wouldn't be better to have it component-specific?

[achmelo]

Do we want to have the same heap size for all components? Wouldn't be better to have it component-specific?

It is per component, e.g. ZWE_configs_heap_max is components.gateway.heap.max in zowe.yaml for gateway, if you want to read this value in discovery you would need ZWE_components_gateway_heap_max

[JirkaAichler]

Do we want to have the same heap size for all components? Wouldn't be better to have it component-specific?

It is per component, e.g. ZWE_configs_heap_max is components.gateway.heap.max in zowe.yaml for gateway, if you want to read this value in discovery you would need ZWE_components_gateway_heap_max

Thank you for explanation

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

The version of Java (11.0.20) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here