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

WT-3959 Manage recovery timestamp via a new schema entry #4043

Merged
merged 14 commits into from Apr 12, 2018

Conversation

agorrod
Copy link
Member

@agorrod agorrod commented Apr 12, 2018

This branch was merged once, but introduced test failures, so I reverted.

@agorrod
Copy link
Member Author

agorrod commented Apr 12, 2018

I've reproduced the failure once in about 30 minutes running 4 reproducers in parallel. The configuration I was using is:

../configure --enable-python --with-spinlock=pthread_adaptive --enable-strict && make -j12
export WIREDTIGER_CONFIG="checkpoint_sync=0,transaction_sync=(method=none)"
for i in `seq 100`; do ./test_timestamp_abort -h WT_TEST.2 -T 5 -t 20 || break ; done

i.e: a slightly longer runtime than the configuration that failed in Jenkins.

@agorrod
Copy link
Member Author

agorrod commented Apr 12, 2018

It reproduced much faster running 20 in parallel:

#!/bin/bash
for j in `seq 100`; do
       	for i in `seq 20`; do
	       	./test_timestamp_abort -h WT_TEST.$i -T 5 -t 10 || break > run.$i.out &
       	done
       	wait
done

michaelcahill and others added 2 commits April 12, 2018 08:38
…a. (#4044)

* WT-3959 Read the recovery timestamp after rolling forward the metadata.

* Shuffle the calling order once more.

We need to set the timestamp even if recovery doesn't do anything.
@sueloverso sueloverso merged commit ae3414b into develop Apr 12, 2018
@michaelcahill michaelcahill deleted the wt-3959-recovery-schema branch April 12, 2018 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants