Alexey Serbin has posted comments on this change. Change subject: [jepen.kudu] some updates ......................................................................
Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/5500/2//COMMIT_MSG Commit Message: PS2, Line 7: epen > jepsen Done http://gerrit.cloudera.org:8080/#/c/5500/2/java/kudu-jepsen/src/main/clojure/jepsen/kudu/nemesis.clj File java/kudu-jepsen/src/main/clojure/jepsen/kudu/nemesis.clj: PS2, Line 24: m > Ah ok - if this is consistent with Jepsen then that's fine with me. After some research I can say that the way it's written here is the default clojure style -- one can find that in core clojure functions and other references such as 'Clojure for the brave and true' at http://www.braveclojure.com/ As for repl, that the snippet: aserbin-MBP:kudu aserbin$ lein repl nREPL server started on port 57993 on host 127.0.0.1 - nrepl://127.0.0.1:57993 REPL-y 0.3.7, nREPL 0.2.12 Clojure 1.8.0 Java HotSpot(TM) 64-Bit Server VM 1.8.0_102-b14 Docs: (doc function-name-here) (find-doc "part-of-name-here") Source: (source function-name-here) Javadoc: (javadoc java-object-or-class-here) Exit: Control+D or (exit) or (quit) Results: Stored in vars *1, *2, *3, an exception in *e user=> (source clojure.test/run-tests) (defn run-tests "Runs all tests in the given namespaces; prints results. Defaults to current namespace if none given. Returns a map summarizing test results." {:added "1.1"} ([] (run-tests *ns*)) ([& namespaces] (let [summary (assoc (apply merge-with + (map test-ns namespaces)) :type :summary)] (do-report summary) summary))) nil user=> -- To view, visit http://gerrit.cloudera.org:8080/5500 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iea1903fae6c2f0371091c773c5ae9b0abbbfbe61 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
