ML: Issue 3565

[rhoope]

Fix to allow user-supplied MPI_Comm in the C-API.

@trilinos/
Request review by @csiefer2

Description

The C-API for MLCreate was hard-coded to use MPI_COMM_WORLD which breaks MPMD uses.

Motivation and Context

Closes Issue #3565

How Has This Been Tested?

I ran the application code in MPMD code to verify the previous error is no longer encountered.

Checklist

• [ x] My commit messages mention the appropriate GitHub issue numbers.
• [x ] My code follows the code style of the affected package(s).
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the code contribution guidelines for this project.
• I have added tests to cover my changes.
• All new and existing tests passed.
• [x ] No new compiler warnings were introduced.
• These changes break backwards compatibility.

[trilinos-autotester]

Status Flag 'Pre-Test Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
NO REVIEWS HAVE BEEN PERFORMED ON THIS PULL REQUEST!

[trilinos-autotester]

Status Flag 'Pre-Test Inspection' - SUCCESS: The last commit to this Pull Request has been INSPECTED AND APPROVED by [ csiefer2 ]!

[trilinos-autotester]

Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects:

Pull Request Auto Testing STARTING (click to expand)

Build Information

Test Name: Trilinos_pullrequest_intel_17.0.1

• Build Num: 1410
• Status: STARTED

Jenkins Parameters

Parameter Name	Value
PULLREQUESTNUM	3656
TEST_REPO_ALIAS	TRILINOS
TRILINOS_SOURCE_BRANCH	ml_issue3565
TRILINOS_SOURCE_REPO	https://github.com/rhoope/Trilinos
TRILINOS_SOURCE_SHA	c17aa9e
TRILINOS_TARGET_BRANCH	develop
TRILINOS_TARGET_REPO	https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA	8d56e74

Build Information

Test Name: Trilinos_pullrequest_gcc_4.9.3

• Build Num: 1973
• Status: STARTED

Jenkins Parameters

Parameter Name	Value
COMPILER_MODULE	sems-gcc/4.9.3
JENKINS_BUILD_TYPE	Release
JENKINS_COMM_TYPE	MPI
JENKINS_DO_COMPLEX	OFF
JENKINS_JOB_TYPE	Experimental
MPI_MODULE	sems-openmpi/1.8.7
PULLREQUESTNUM	3656
TEST_REPO_ALIAS	TRILINOS
TRILINOS_SOURCE_BRANCH	ml_issue3565
TRILINOS_SOURCE_REPO	https://github.com/rhoope/Trilinos
TRILINOS_SOURCE_SHA	c17aa9e
TRILINOS_TARGET_BRANCH	develop
TRILINOS_TARGET_REPO	https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA	8d56e74

Build Information

Test Name: Trilinos_pullrequest_gcc_4.8.4

• Build Num: 1635
• Status: STARTED

Jenkins Parameters

Parameter Name	Value
COMPILER_MODULE	sems-gcc/4.8.4
JENKINS_BUILD_TYPE	Release
JENKINS_COMM_TYPE	MPI
JENKINS_DO_COMPLEX	OFF
JENKINS_JOB_TYPE	Experimental
MPI_MODULE	sems-openmpi/1.8.7
PULLREQUESTNUM	3656
TEST_REPO_ALIAS	TRILINOS
TRILINOS_SOURCE_BRANCH	ml_issue3565
TRILINOS_SOURCE_REPO	https://github.com/rhoope/Trilinos
TRILINOS_SOURCE_SHA	c17aa9e
TRILINOS_TARGET_BRANCH	develop
TRILINOS_TARGET_REPO	https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA	8d56e74

Using Repos:

Repo: TRILINOS (rhoope/Trilinos)

• Branch: ml_issue3565
• SHA: c17aa9e
• Mode: TEST_REPO

Pull Request Author: rhoope

[trilinos-autotester]

Status Flag 'Pull Request AutoTester' - Jenkins Testing: all Jobs PASSED

Pull Request Auto Testing has PASSED (click to expand)

Build Information

Test Name: Trilinos_pullrequest_intel_17.0.1

• Build Num: 1410
• Status: PASSED

Jenkins Parameters

Parameter Name	Value
PULLREQUESTNUM	3656
TEST_REPO_ALIAS	TRILINOS
TRILINOS_SOURCE_BRANCH	ml_issue3565
TRILINOS_SOURCE_REPO	https://github.com/rhoope/Trilinos
TRILINOS_SOURCE_SHA	c17aa9e
TRILINOS_TARGET_BRANCH	develop
TRILINOS_TARGET_REPO	https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA	8d56e74

Build Information

Test Name: Trilinos_pullrequest_gcc_4.9.3

• Build Num: 1973
• Status: PASSED

Jenkins Parameters

Parameter Name	Value
COMPILER_MODULE	sems-gcc/4.9.3
JENKINS_BUILD_TYPE	Release
JENKINS_COMM_TYPE	MPI
JENKINS_DO_COMPLEX	OFF
JENKINS_JOB_TYPE	Experimental
MPI_MODULE	sems-openmpi/1.8.7
PULLREQUESTNUM	3656
TEST_REPO_ALIAS	TRILINOS
TRILINOS_SOURCE_BRANCH	ml_issue3565
TRILINOS_SOURCE_REPO	https://github.com/rhoope/Trilinos
TRILINOS_SOURCE_SHA	c17aa9e
TRILINOS_TARGET_BRANCH	develop
TRILINOS_TARGET_REPO	https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA	8d56e74

Build Information

Test Name: Trilinos_pullrequest_gcc_4.8.4

• Build Num: 1635
• Status: PASSED

Jenkins Parameters

Parameter Name	Value
COMPILER_MODULE	sems-gcc/4.8.4
JENKINS_BUILD_TYPE	Release
JENKINS_COMM_TYPE	MPI
JENKINS_DO_COMPLEX	OFF
JENKINS_JOB_TYPE	Experimental
MPI_MODULE	sems-openmpi/1.8.7
PULLREQUESTNUM	3656
TEST_REPO_ALIAS	TRILINOS
TRILINOS_SOURCE_BRANCH	ml_issue3565
TRILINOS_SOURCE_REPO	https://github.com/rhoope/Trilinos
TRILINOS_SOURCE_SHA	c17aa9e
TRILINOS_TARGET_BRANCH	develop
TRILINOS_TARGET_REPO	https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA	8d56e74

CDash Test Results for PR# 3656.

[trilinos-autotester]

Status Flag 'Pre-Merge Inspection' - SUCCESS: The last commit to this Pull Request has been INSPECTED AND APPROVED by [ csiefer2 ]!

[trilinos-autotester]

Status Flag 'Pull Request AutoTester' - AutoMerge IS ENABLED, but the Label AT: AUTOMERGE is not set. Either set Label AT: AUTOMERGE or manually merge the PR...