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

Modify behaviour so that failing to provide a JVM server when required generates an error #138

Closed
davenice opened this issue Jan 6, 2020 · 5 comments · Fixed by #143
Closed
Labels
enhancement New feature or request usability The tool could be easier to use

Comments

@davenice
Copy link
Contributor

davenice commented Jan 6, 2020

Currently we default defaultjvmserver to MYJVMS. However there's no sensible default JVM server name. If users are bundling a Java app, they need to specify a JVM server name.

We should alter the behaviour so that if a JVM server name isn't provided either via defaultjvmserver or specific JVM servers on every relevant project, the build fails and the user is warned.

The only people this will affect is those who genuinely are happy with the default because they have a JVM server called MYJVMS .... it seems that this group will be vanishingly small!

@davenice davenice added enhancement New feature or request usability The tool could be easier to use labels Jan 6, 2020
@IvanHargreaves
Copy link
Member

Hi Dave, DFHWLP might be a sensible default. That is the value used in the Liberty JVM server samples and is shipped with CICS in the pre-defined RDO groups. That said, a warning might make it more obvious that the user hasn't consciously set it. Resolution of such a warning might be to help populate the field with a value of DFHWLP rather than offering blank.

@davenice
Copy link
Contributor Author

davenice commented Jan 6, 2020

Thanks Ivan - yes we discussed DFHWLP as a default but decided that really if the user was doing Java they needed to set the value to something sensible.

We couldn't get consensus that a majority of users would use DFHWLP a majority of the time, and because of this, and the simplicity of resolving at build time compared to runtime, we opted for forcing the user to specify.

I don't think there will be a way to automatically populate fields because we don't have a custom editor here. However using DFHWLP in our samples could be a better approach than MYJVMS.

@davenice
Copy link
Contributor Author

davenice commented Jan 6, 2020

Had a further chat with Ivan - we reckon the probability is that a new user would use DFHWLP, customising the name is more advanced.

On that basis we think the ideal is to have an error message that tells them to specify a JVM server (e.g. DFHWLP). If that's hard to do then just updating the default is likely to help a bunch of people who are trying to get started.

@davenice
Copy link
Contributor Author

davenice commented Jan 6, 2020

Also we should update our samples to use DFHWLP and DFHOSGI as better suggestions.

@ledina
Copy link
Member

ledina commented Feb 27, 2020

The JVM server rename is being tracked under #132

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request usability The tool could be easier to use
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants