-
Notifications
You must be signed in to change notification settings - Fork 845
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
java: catch exceptions if we fail to load mpi_java #10697
Conversation
a step along the way to retaining ease of use with prrte for java apps |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hppritcha Just a question. How the behavior is changed? Only the messsage change? Exiting cleanly?
ompi/mpi/java/java/MPI.java
Outdated
{ | ||
System.loadLibrary("mpi_java") ; | ||
} | ||
catch (UnsatisfiedLinkError e) | ||
{ | ||
System.err.println("mpi java lib failed to load" + e + "\n") ; | ||
System.exit(1) ; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use tabs instead of spaces for indentation. Other Java codes in OMPI use tabs.
ompi/mpi/java/java/MPI.java
Outdated
} | ||
catch (UnsatisfiedLinkError e) | ||
{ | ||
System.err.println("mpi java lib failed to load" + e + "\n") ; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please insert a space of something before the Java exception message.
System.err.println("mpi java lib failed to load" + e + "\n") ; | |
System.err.println("mpi java lib failed to load: " + e + "\n") ; |
bc3bac9
to
4fa49cb
Compare
@kawashima-fj check now, hopefully i got the tabs right! |
ompi/mpi/java/java/MPI.java
Outdated
try | ||
{ | ||
System.loadLibrary("mpi_java") ; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The line 210 still has space-indentation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
okay i think that fixed it.
library. Signed-off-by: Howard Pritchard <howardp@lanl.gov>
4fa49cb
to
54042fc
Compare
@kawashima-fj i think line 210 is fixed now |
library.
Signed-off-by: Howard Pritchard howardp@lanl.gov