Skip to content

Conversation

bosilca
Copy link
Member

@bosilca bosilca commented Aug 14, 2019

Fixes a datatype issue in ompi-test/mpi2basic-tests/filetests/fileview.c reported by Edgar.

In addition to the fix for the reported issue, this patch contains a renaming of the OPAL datatype internal variables used for the datatype debugging infrastructure.

Needs to be included in #6863

Fixes the convertor iovec description on the MPI-IO reported by Edgar.

Signed-off-by: George Bosilca <[email protected]>
@bosilca bosilca requested a review from edgargabriel August 14, 2019 05:20
No code or logic changes.

Signed-off-by: George Bosilca <[email protected]>
Signed-off-by: Jeff Squyres <[email protected]>
@jsquyres
Copy link
Member

I separated out the unrelated whitespace-only file change into its own commit. Because that's how I roll.

Copy link
Member

@edgargabriel edgargabriel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bosilca, looks good, thanks! I can confirm that my tests are passing now again.

@bosilca bosilca merged commit 190d5f1 into open-mpi:master Aug 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants