nathanielherman / sto

Software Transactional Objects
20 stars 12 forks source link

trans_test currently fails commit_tid() assert with CONSISTENCY_CHECK enabled #18

Closed tslilyai closed 6 years ago

tslilyai commented 6 years ago

(Hi everyone!)

ghost commented 6 years ago

Yes this has long been a problem for trans_test because its way of using commit_tid() is really out of the scope of the STO design. I think it may be better to add another state/another commit_tid() method to handle this situation because this assert is still somewhat useful while not using trans_test. Is CONSISTENCY_CHECK only used by trans_test?

tslilyai commented 6 years ago

Yeah, it seems like trans_test (on the master branch at least) is the only thing that uses consistency check.

On Oct 7, 2017 06:11, "Yihe (William) Huang" notifications@github.com wrote:

Yes this has long been a problem for trans_test because its way of using commit_tid() is really out of the scope of the STO design. I think it may be better to add another state/another commit_tid() method to handle this situation because this assert is still somewhat useful while not using trans_test. Is CONSISTENCY_CHECK only used by trans_test?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/nathanielherman/sto/pull/18#issuecomment-334908951, or mute the thread https://github.com/notifications/unsubscribe-auth/AIpSfsgOXj55bqASRkAaSY1Ol9m-k4MWks5spvnTgaJpZM4PxN_r .

dimstamat commented 6 years ago

Hi everyone,

Do you know if there in an API documentation for STO, or any examples of STO usage other than the benchmarks? I am working on extending STO to support layout modifications and it would be useful to run some simple user programs using STO and see what is happening underneath.

Many thanks, Dimos Stamatakis

On Oct 7, 2017, at 9:27 PM, Yihe (William) Huang notifications@github.com wrote:

Merged #18 https://github.com/nathanielherman/sto/pull/18.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/nathanielherman/sto/pull/18#event-1283041500, or mute the thread https://github.com/notifications/unsubscribe-auth/AIBLBjg7qiiO0UsTUQmHJTze2g13JFycks5sqCUbgaJpZM4PxN_r.