Skip to content
This repository has been archived by the owner on Sep 27, 2019. It is now read-only.

Skip adding READs to the RWSet #1425

Merged
merged 2 commits into from
Jun 24, 2018
Merged

Skip adding READs to the RWSet #1425

merged 2 commits into from
Jun 24, 2018

Conversation

mbutrovich
Copy link
Contributor

Addresses point 2 in #1420. There's no reason to add READs to the RWSet in MVTO.

@mbutrovich mbutrovich self-assigned this Jun 22, 2018
@coveralls
Copy link

coveralls commented Jun 22, 2018 •
edited
Loading

Coverage Status

Coverage increased (+0.04%) to 77.89% when pulling 7b78404 on mbutrovich:skip_reads into 55abc0c on cmu-db:master.

tli2
tli2 approved these changes Jun 22, 2018
View reviewed changes
Copy link
Contributor

@tli2 tli2 left a comment

Choose a reason for hiding this comment

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

LGTM

@mbutrovich mbutrovich mentioned this pull request Jun 22, 2018
Open
@tli2 tli2 merged commit 29f92eb into cmu-db:master Jun 24, 2018
@mbutrovich mbutrovich deleted the skip_reads branch June 25, 2018 14:46
mtunique pushed a commit to mtunique/peloton that referenced this pull request Apr 16, 2019
@mbutrovich @tli2
Skip recording READs in the RWSet. (cmu-db#1425)
649d972
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@tli2 tli2 tli2 approved these changes

Assignees

@mbutrovich mbutrovich

Labels
ready_for_review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mbutrovich @coveralls @tli2