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.

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

UCT/IB: Use class infrastructure for IB objects. #28

Merged
merged 3 commits into from
Nov 24, 2014

Conversation

yosefe
Copy link
Contributor

@yosefe yosefe commented Nov 24, 2014

This 2 commits add a class infrastructure to UCX. The big difference from MCA is that it supports a constructor failure, an automatically cleans-up partial initialization.
the plan is to also add logging/to_string logic to classes.

@mellanox-github
Copy link
Contributor

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://bgate.mellanox.com:8000/jenkins-secure/job/gh-ucx-pr/50/
Test PASSed.

@yosefe
Copy link
Contributor Author

yosefe commented Nov 24, 2014

retest this please

@mellanox-github
Copy link
Contributor

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://bgate.mellanox.com:8000/jenkins-secure/job/gh-ucx-pr/51/
Test PASSed.

@yosefe
Copy link
Contributor Author

yosefe commented Nov 24, 2014

retest this please

@mellanox-github
Copy link
Contributor

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://bgate.mellanox.com:8000/jenkins-secure/job/gh-ucx-pr/52/
Test PASSed.

@mellanox-github
Copy link
Contributor

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://bgate.mellanox.com:8000/jenkins-secure/job/gh-ucx-pr/53/
Test PASSed.

@shamisp
Copy link
Contributor

shamisp commented Nov 24, 2014

👍

shamisp pushed a commit that referenced this pull request Nov 24, 2014
UCT/IB: Use class infrastructure for IB objects.
@shamisp shamisp merged commit c23b500 into openucx:master Nov 24, 2014
@yosefe yosefe deleted the topic/add-class-infra branch November 25, 2014 17:50
amaslenn pushed a commit to amaslenn/ucx that referenced this pull request Jun 2, 2020
shizhibao pushed a commit to shizhibao/ucx that referenced this pull request Jan 16, 2021
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