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

QA-3909 Append scenario for AI3 #4

Merged
merged 30 commits into from
Nov 7, 2023
Merged

QA-3909 Append scenario for AI3 #4

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
89271d0
Append test, not working yet
ahitrin Oct 30, 2023
3eeba16
Avoid infinite print
ahitrin Nov 1, 2023
01b9d97
Do not shadow names like "test" and "key"
ahitrin Nov 1, 2023
9be060c
Rebalance open! and setup! methods
ahitrin Nov 1, 2023
ed2da47
Stupid value insert (ignoring jepsen data for now)
ahitrin Nov 1, 2023
1aee052
wip
ahitrin Nov 2, 2023
71f6a7d
wip
ahitrin Nov 2, 2023
dadb739
extract
ahitrin Nov 2, 2023
918849f
fixes
ahitrin Nov 2, 2023
0371300
convert result into list
ahitrin Nov 2, 2023
13a5d7b
insert or update
ahitrin Nov 2, 2023
b520274
destructure
ahitrin Nov 2, 2023
0053c29
simplify
ahitrin Nov 2, 2023
33aa17a
fix
ahitrin Nov 2, 2023
981fd18
Add pseudo-nemesis for append test
ahitrin Nov 2, 2023
d02caf7
Comment out debug logging
ahitrin Nov 2, 2023
f75442f
cleanup
ahitrin Nov 3, 2023
bfc7ff5
cleanup imports
ahitrin Nov 3, 2023
aff40a8
extract run-sql function
ahitrin Nov 3, 2023
fac6e10
Try to setup consistency models
ahitrin Nov 3, 2023
aa0694c
Remove unused pds settings
ahitrin Nov 3, 2023
2c9d4b1
convert long expression into several steps
ahitrin Nov 3, 2023
c0d499a
Fix: do not use test name "basic-test"
ahitrin Nov 3, 2023
2224265
Explicitly open transaction for read
ahitrin Nov 3, 2023
1703392
simplify
ahitrin Nov 3, 2023
4961395
Convert ignite3/generator into ignite3/wrap-generator
ahitrin Nov 6, 2023
2484356
Fix: make append test finishable
ahitrin Nov 6, 2023
4a396bc
Fixes: use txn to read, return :process
ahitrin Nov 6, 2023
89992c8
Fix hostory creation: use assoc to keep all fields needed
ahitrin Nov 6, 2023
05621ea
Remove temporary nemesis implementation
ahitrin Nov 7, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add pseudo-nemesis for append test
  • Loading branch information
@ahitrin
ahitrin committed Nov 2, 2023
commit 981fd18e8b3f0de453e8f58e2c21a0cd68109467
13 changes: 12 additions & 1 deletion ignite-3/src/jepsen/ignite3/append.clj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
[jepsen [checker :as checker]
[client :as client]
[ignite3 :as ignite3]
[independent :as independent]]
[nemesis :as nemesis]]
[jepsen.checker.timeline :as timeline]
[jepsen.tests.cycle.append :as app]
[knossos.model :as model])
Expand Down Expand Up @@ -86,6 +86,17 @@
(close! [this test]
(.close ignite)))

(def pseudo-noop
"Does nothing but logging."
(reify nemesis/Nemesis
(setup! [this test] this)
(invoke! [this test op]
(log/info "Nemesis received" op)
(assoc op :type :info))
(teardown! [this test] this)
nemesis/Reflection
(fs [this] #{})))

(comment "for repl"

(def c (client/open! (Client. nil) {} "127.0.0.1"))
Expand Down
3 changes: 2 additions & 1 deletion ignite-3/src/jepsen/ignite3/runner.clj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@
"register" register/register-test})

(def nemesis-types
{"noop" jepsen.nemesis/noop})
{"noop" jepsen.nemesis/noop
"append" append/pseudo-noop})

(def opt-spec
"Command line options for tools.cli"
Expand Down