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

Remove RWlock in PMGDQueryHandler #50

Merged
merged 1 commit into from
Dec 21, 2018
Merged

Remove RWlock in PMGDQueryHandler #50

merged 1 commit into from
Dec 21, 2018

Conversation

luisremis
Copy link
Contributor

This will be merged once the concurrency branch is merged into develop for PMGD.
Working, all tests passing.

@luisremis luisremis requested a review from vishakha041 November 6, 2018 07:28
vishakha041
vishakha041 approved these changes Nov 7, 2018
View reviewed changes
Copy link
Contributor

@vishakha041 vishakha041 left a comment

Choose a reason for hiding this comment

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

Make sure to remove the RWLock.h file added with the lock commit.
And how do we make sure that this commit onwards we only use concurrent PMGD?

@luisremis
Copy link
Contributor Author

Concurrency branch in PMGD merged in to develop, we are good to go here.

@luisremis luisremis merged commit 1983062 into develop Dec 21, 2018
@luisremis luisremis deleted the remove_rwlock branch May 3, 2019 23:16
Ragaad pushed a commit to Ragaad/vdms that referenced this pull request Jun 16, 2022
@cwlacewe
Update INSTALL.md (IntelLabs#50)
eb34f29 
* Remove unused commons-codec-1.15 from dockerfiles and updated INSTALL.md to match
* Sort packages and remove duplicates for dockerfiles and INSTALL.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vishakha041 vishakha041 vishakha041 approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@luisremis @vishakha041