David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9526 )
Change subject: [WIP] updated to use READ_YOUR_WRITES scan mode for Jepsen tests ...................................................................... Patch Set 1: (5 comments) we'll also need to test RYR, which I'm not sure the register test can do, since it only writes a single row. cockroachdb has a good set of tests that we might want to look at: https://github.com/jepsen-io/jepsen/tree/master/cockroachdb http://gerrit.cloudera.org:8080/#/c/9526/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/9526/1//COMMIT_MSG@9 PS1, Line 9: This patches updated the Jespen tests to use READ_YOUR_WRITES scan mode. we're supposed to have both eventually right? http://gerrit.cloudera.org:8080/#/c/9526/1//COMMIT_MSG@13 PS1, Line 13: I ran the Jepesen test with this change and it passed. the test only failed very occasionally, we'd need quite a few runs before we're confident the problem doens't reappear http://gerrit.cloudera.org:8080/#/c/9526/1//COMMIT_MSG@14 PS1, Line 14: https://drive.google.com/file/d/1jW1CeGjB9AdnJZ2XbhJrf0IyOwXnKC_p/view?usp=sharing thanks for sharing this. http://gerrit.cloudera.org:8080/#/c/9526/1/java/kudu-jepsen/src/main/clojure/jepsen/kudu/register.clj File java/kudu-jepsen/src/main/clojure/jepsen/kudu/register.clj: http://gerrit.cloudera.org:8080/#/c/9526/1/java/kudu-jepsen/src/main/clojure/jepsen/kudu/register.clj@36 PS1, Line 36: [table-created? kclient ktable] : (reify jepsen.client/Client : (setup! [_ test _] : "Create the client and the test table. Use the same Kudu client instance " : "across all test actors to achieve timestamp propagation for all " : "operations." : (let [kclient (kc/sync-client (:master-addresses test)) : table-name (:table-name test) : ktable (locking table-created? : (when (compare-and-set! table-created? false true) : (kc/create-table : kclient : table-name : kt/kv-table-schema : (let [ranges (:table-ranges test) : rep-factor (:num-replicas test)] : (if (nil? ranges) : (kt/kv-table-options-hash rep-factor (count (:tservers test))) : (kt/kv-table-options-range rep-factor ranges))))) : (kc/open-table kclient table-name))] : (client table-created? kclient ktable))) this is just a revert of the changes we made to have 1 client per table, vs one per thread. eventually we'll want to be able to choose so that we test both modes. http://gerrit.cloudera.org:8080/#/c/9526/1/java/kudu-jepsen/src/main/clojure/jepsen/kudu/table.clj File java/kudu-jepsen/src/main/clojure/jepsen/kudu/table.clj: http://gerrit.cloudera.org:8080/#/c/9526/1/java/kudu-jepsen/src/main/clojure/jepsen/kudu/table.clj@100 PS1, Line 100: READ_YOUR_WRITES this will have to be a parameter -- To view, visit http://gerrit.cloudera.org:8080/9526 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I92d5c0e3b91af58576eb6cd408d922ec7c0fef6c Gerrit-Change-Number: 9526 Gerrit-PatchSet: 1 Gerrit-Owner: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Wed, 07 Mar 2018 07:26:15 +0000 Gerrit-HasComments: Yes