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-4388 Fixes in append scenario #5

Merged
merged 19 commits into from
Nov 13, 2023
Merged

QA-4388 Fixes in append scenario #5

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
a82c287
Micro: use case instead of if when choosing action
ahitrin Nov 7, 2023
dee5b6d
split long function into several smaller ones
ahitrin Nov 7, 2023
16d9ef3
More straightforward list creation
ahitrin Nov 7, 2023
d5dbcdf
Mark args as unused
ahitrin Nov 7, 2023
77940f5
Implement retries, attempt 1/10
ahitrin Nov 7, 2023
e0f32a7
Fix: avoid NumberFormatException on missing values
ahitrin Nov 7, 2023
c603fce
Raise max attempts to 20
ahitrin Nov 7, 2023
b943f2b
Retry on replication timeout too
ahitrin Nov 7, 2023
341f4b9
format
ahitrin Nov 7, 2023
bbf7e49
Replace selector fn with a simple map
ahitrin Nov 8, 2023
c437094
Wait before retry
ahitrin Nov 8, 2023
373dd71
Refresh comments
ahitrin Nov 8, 2023
4121d20
Fix catching order
ahitrin Nov 8, 2023
7add130
Print table content at the end of the test
ahitrin Nov 8, 2023
82a74b6
Process exceptions in the same way, again
ahitrin Nov 8, 2023
b3e2763
Revert "Process exceptions in the same way, again"
ahitrin Nov 8, 2023
9cc413f
TransactionException is just an info, okay
ahitrin Nov 8, 2023
a4979d3
Try to use a single transaction for all ops (no success yet)
ahitrin Nov 9, 2023
643bd05
Fix: eagerly invoke transaction ops
ahitrin Nov 9, 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
Refresh comments
  • Loading branch information
@ahitrin
ahitrin committed Nov 8, 2023
commit 373dd71fa5224c61813ab7629b0b74c47c105366
4 changes: 1 addition & 3 deletions ignite-3/src/jepsen/ignite3/append.clj
View file
Original file line number Diff line number Diff line change
@@ -92,7 +92,6 @@
client/Client
;
(open! [this test node]
; (log/info "Node: " node)
(let [ignite (.build (.addresses (IgniteClient/builder) (into-array [(str node ":10800")])))]
(assoc this :ignite ignite)))
;
@@ -103,13 +102,12 @@
(log/info "Table" table-name "created")))
;
(invoke! [this test op]
; (log/info "Received: " op)
(let [ops (:value op)
; result (map #(invoke-op ignite %) ops)
result (map #(invoke-with-retries ignite %) ops)
overall-result (assoc op
:type :info
:value (into [] result))]
; (log/info "Returned: " overall-result)
overall-result))
;
(teardown! [this test])