Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Implement support for big I/O for independent reads #1446

Merged
merged 3 commits into from
Feb 18, 2022
Merged

Implement support for big I/O for independent reads #1446

merged 3 commits into from
Feb 18, 2022

Conversation

fortnern
Copy link
Member

Also add test case

jhendersonHDF
jhendersonHDF approved these changes Feb 17, 2022
View reviewed changes
Copy link
Collaborator

@jhendersonHDF jhendersonHDF left a comment

Choose a reason for hiding this comment

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

LGTM. Ideally, H5FD__mpio_write would be updated to mirror the structure of this code, but I don't think it's a big deal for this PR.

@lrknox lrknox merged commit feda52b into HDFGroup:develop Feb 18, 2022
jhendersonHDF pushed a commit to jhendersonHDF/hdf5 that referenced this pull request May 3, 2022
@fortnern @github-actions @jhendersonHDF
Implement support for big I/O for independent reads (HDFGroup#1446)
9156584 
* Implement big I/O support for independent reads. Add test for this.

* Committing clang-format changes

* Update big I/O code in H5FD__mpio_write() to match that in
H5FD__mpio_read().

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
derobins pushed a commit that referenced this pull request May 3, 2022
@jhendersonHDF @fortnern @github-actions
Implement support for big I/O for independent reads (#1446) (#1719)
a1b8141 
* Implement big I/O support for independent reads. Add test for this.

* Committing clang-format changes

* Update big I/O code in H5FD__mpio_write() to match that in
H5FD__mpio_read().

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

Co-authored-by: Neil Fortner <nfortne2@hdfgroup.org>
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@epourmal epourmal epourmal approved these changes

@jhendersonHDF jhendersonHDF jhendersonHDF approved these changes

@qkoziol qkoziol Awaiting requested review from qkoziol qkoziol is a code owner

@soumagne soumagne Awaiting requested review from soumagne

@brtnfld brtnfld Awaiting requested review from brtnfld brtnfld is a code owner

@ChristopherHogan ChristopherHogan Awaiting requested review from ChristopherHogan

@gnuoyd gnuoyd Awaiting requested review from gnuoyd

@jrmainzer jrmainzer Awaiting requested review from jrmainzer

@vchoi-hdfgroup vchoi-hdfgroup Awaiting requested review from vchoi-hdfgroup vchoi-hdfgroup is a code owner

@derobins derobins Awaiting requested review from derobins derobins is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@fortnern @epourmal @jhendersonHDF @lrknox