Skip to content

Moved HDF5 collective_md option into the backend. #94

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

Closed
wants to merge 1 commit into from

Conversation

JulianKunkel
Copy link
Collaborator

@roblatham00 @glennklockwood
This is an example how the backend-specific options can be used to remove the dependencies to backends from the global options.

Example how to use:
./src/ior -a HDF5 -- --collectiveMetadata
or
./src/ior -a HDF5 -- -c

Just to wrap up the discussion.

What is missing is that these options can be also controlled from inside an IOR script (right now only parsed once in the command line). That could be e.g.: backend_collectiveMetadata (for simplicity)
If nobody disagrees, I will move along at some point and make this happen. I would love to get rid of the plugin dependencies from ior.h

@glennklockwood
Copy link
Contributor

This may make the code a little cleaner, but it goes against the convention of what -- means in the rest of the GNU userspace and is going to cause a lot of confusion. I'm not too keen on having two separate "namespaces" for options exposed all the way to users.

My preference would be to keep the user interface clean and be clever in the code because we have far more IOR users than we have developers. I had envisioned just allowing option parsers to exist inside the AIORIs, and any options that are rejected by the main ior options parser then get passed down to the backends. This allows backends to be self-contained a little better without forcing the burden on users.

@JulianKunkel
Copy link
Collaborator Author

JulianKunkel commented Oct 2, 2018 via email

@roblatham00
Copy link
Contributor

roblatham00 commented Oct 2, 2018 via email

@JulianKunkel
Copy link
Collaborator Author

JulianKunkel commented Oct 2, 2018 via email

@jti-lanl
Copy link
Contributor

jti-lanl commented Oct 2, 2018 via email

@JulianKunkel
Copy link
Collaborator Author

JulianKunkel commented Oct 2, 2018 via email

@johnbent
Copy link
Collaborator

johnbent commented Oct 3, 2018 via email

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.

5 participants