Skip to content
This repository was archived by the owner on Apr 1, 2023. It is now read-only.

One hot transaction#213

Draft
ammuv wants to merge 6 commits into
masterfrom
one-hot-transaction
Draft

One hot transaction#213
ammuv wants to merge 6 commits into
masterfrom
one-hot-transaction

Conversation

@ammuv

@ammuv ammuv commented Aug 18, 2021

Copy link
Copy Markdown
Contributor
  • Added one hot transaction feature to K23SI client and server.
  • Modified YCSB tests to use one hot transactions.

@ammuv ammuv requested review from iafuture and jfunston and removed request for jfunston August 18, 2021 17:29

@iafuture iafuture left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Initially i thought it was a typo, but what is "one hot txn"? I think it should be "one-shot" or "one-hop"

@ammuv

ammuv commented Aug 18, 2021

Copy link
Copy Markdown
Contributor Author

Initially i thought it was a typo, but what is "one hot txn"? I think it should be "one-shot" or "one-hop"

Oh...I thought it was one-hot the whole time. Like they are "hot" transactions or transactions that are quicker because commits need not be sent by client. But now that you say, "shot" makes more sense!

Comment thread src/k2/module/k23si/client/k23si_client.h Outdated
Comment thread src/k2/module/k23si/client/k23si_client.h Outdated
Comment thread src/k2/module/k23si/client/k23si_client.h Outdated
Comment thread src/k2/module/k23si/client/k23si_client.cpp
Comment thread src/k2/module/k23si/Module.cpp Outdated
Comment thread src/k2/module/k23si/Module.cpp Outdated
Comment thread src/k2/module/k23si/Module.cpp Outdated
Comment thread src/k2/module/k23si/Module.cpp Outdated
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants