David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9235 )
Change subject: docs: improvements to transaction semantics ...................................................................... Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/9235/1/docs/transaction_semantics.adoc File docs/transaction_semantics.adoc: http://gerrit.cloudera.org:8080/#/c/9235/1/docs/transaction_semantics.adoc@59 PS1, Line 59: with different levels of consistency or correctness guarantees. Scans can perform time-travel reads, i.e. nit: is this over the col limit? (seems like it, but i might be wrong as I forget if we have one for docs or what it is) http://gerrit.cloudera.org:8080/#/c/9235/1/docs/transaction_semantics.adoc@181 PS1, Line 181: `READ_LATEST`:: This is the default read mode. The server takes a snapshot of : the MVCC state and proceeds with the read immediately. Reads in this mode only yield : 'Read Committed' isolation. are you leaving the new mode out on purpose? http://gerrit.cloudera.org:8080/#/c/9235/1/docs/transaction_semantics.adoc@205 PS1, Line 205: Note don't we have a special "note" marker, or am I mistaken? http://gerrit.cloudera.org:8080/#/c/9235/1/docs/transaction_semantics.adoc@222 PS1, Line 222: Writes ahah, well caught :) -- To view, visit http://gerrit.cloudera.org:8080/9235 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I23a2751923a4214f52e6cdb233f7e3aeee207da2 Gerrit-Change-Number: 9235 Gerrit-PatchSet: 1 Gerrit-Owner: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Alex Rodoni <arod...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Wed, 21 Feb 2018 01:00:06 +0000 Gerrit-HasComments: Yes
