Skip to content
This repository was archived by the owner on Sep 30, 2022. It is now read-only.

Conversation

@rhc54
Copy link

@rhc54 rhc54 commented Oct 2, 2014

SVN r32817
Correct the classpath to correctly include the local directory so Java programs find the application class

SVN r32823
Cover the remaining code paths for Java apps to define class path

Correct the classpath to correctly include the local directory so Java programs find the application class

SVN r32823
Cover the remaining code paths for Java apps to define class path
@rhc54
Copy link
Author

rhc54 commented Oct 2, 2014

@osvegis - can you please review?

@osvegis
Copy link
Contributor

osvegis commented Oct 2, 2014

reviewed
It's ok. I checked it with and without -cp parameter and CLASSPATH variable.

@rhc54
Copy link
Author

rhc54 commented Oct 2, 2014

RM-approved

rhc54 pushed a commit that referenced this pull request Oct 2, 2014
@rhc54 rhc54 merged commit 2521257 into open-mpi:v1.8 Oct 2, 2014
@jsquyres jsquyres added this to the v1.8.4 milestone Oct 2, 2014
@rhc54 rhc54 self-assigned this Oct 3, 2014
mike-dubman added a commit to mike-dubman/ompi-release that referenced this pull request Oct 28, 2014
…rom_gerrit

Merge from mellanox-v1.8 internal branch
rhc54 pushed a commit that referenced this pull request Oct 31, 2014
jsquyres added a commit that referenced this pull request May 26, 2015
@hppritcha hppritcha mentioned this pull request Aug 26, 2015
jsquyres added a commit that referenced this pull request Aug 31, 2015
Split the list of subroutines into cases #1 and #2, just for clarity.

(cherry picked from commit 596557e)
rhc54 pushed a commit that referenced this pull request Nov 10, 2015
orte proc_info.h: use symbolic names
artpol84 added a commit to artpol84/ompi-release that referenced this pull request Dec 9, 2015
Previous commit f279474 reverts Nathans changes. However it turns out
that I was unable to trace his logic until I started investigation of
icsplit hang. Bug was triggered when splitting Intercom was giving a group
where on side of the communicator was empty (icsplit, intercom create open-mpi#2).
in this case remote_size == 0 and there is no way to distinguish between
inter- and intra-communicator.
Conclusion: We do need to distinguish between intra- and inter-communicators.
So we should use ompi_mpi_group_null.group.

(cherry picked from commit open-mpi/ompi@7690f40)
alinask pushed a commit to alinask/ompi-release that referenced this pull request Dec 10, 2015
Split the list of subroutines into cases open-mpi#1 and open-mpi#2, just for clarity.
alinask pushed a commit to alinask/ompi-release that referenced this pull request Dec 10, 2015
Previous commit f279474 reverts Nathans changes. However it turns out
that I was unable to trace his logic until I started investigation of
icsplit hang. Bug was triggered when splitting Intercom was giving a group
where on side of the communicator was empty (icsplit, intercom create open-mpi#2).
in this case remote_size == 0 and there is no way to distinguish between
inter- and intra-communicator.
Conclusion: We do need to distinguish between intra- and inter-communicators.
So we should use ompi_mpi_group_null.group.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants