hint processing should not open files

[roblatham00]

move opening of files from hint processing and into open routines.

This is MPICH commit 92f1c69f0de8 and 22a77dceda11

more information at https://trac.mpich.org/projects/mpich/ticket/2261
Ref: #158
closes openmpi #158
Signed-off-by: Pavan Balaji balaji@anl.gov

[goodell]

Hey Rob, thanks for the PR, since I know you're under no obligation to help maintain ROMIO in OMPI. I'm going to let people who actually deal with MPI-IO in OMPI look at this, but I have one minor nit:

Ref: #2261

This breadcrumb should probably be removed from the commit before it gets merged, since it refers to an MPICH Trac ticket number, not an OMPI github issue number. At some point when OMPI hits >2261 PRs/issues, this breadcrumb will potentially become confusing.

[roblatham00]

In this case the motivation is more selfish than altruistic: a Darshan user
using OpenMPI couldn't create log files because of this. Editing the
breadcrumb makes sense (hope github really did edit it...), and I'll try to
remember to namespace these identifiers in the future.

On Wed, Jul 29, 2015 at 10:59 AM, Dave Goodell notifications@github.com
wrote:

Hey Rob, thanks for the PR, since I know you're under no obligation to
help maintain ROMIO in OMPI. I'm going to let people who actually deal with
MPI-IO in OMPI look at this, but I have one minor nit:

Ref: #2261

This breadcrumb should probably be removed from the commit before it gets
merged, since it refers to an MPICH Trac ticket number, not an OMPI github
issue number. At some point when OMPI hits >2261 PRs/issues, this
breadcrumb will potentially become confusing.

—
Reply to this email directly or view it on GitHub
#757 (comment).

[hppritcha]

I'd prefer to merge this PR in after we've fixed some lustre problems in master. The problem results both in ompi io and romio components failing on all but trivial tests when using lustre. Once we get lustre working again and can double check this PR, I'm okay with merging it.

[goodell]

(hope github really did edit it...)

Looks like you successfully edited the PR description, but never force-pushed an updated commit with a fixed commit message.

[hppritcha]

I tried this out on a lustre file system now that ompi has gotten a visit from the lustre medic. Works for me.
👍