You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Mike Percy (Code Review)" <ge...@cloudera.org> on 2018/10/11 20:49:09 UTC

[kudu-CR] KUDU-2463 pt 3: don't scan if MVCC hasn't moved

Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/11428 )

Change subject: KUDU-2463 pt 3: don't scan if MVCC hasn't moved
......................................................................


Patch Set 11: Code-Review+2

(1 comment)

looks good just one optional nit

http://gerrit.cloudera.org:8080/#/c/11428/11/src/kudu/tserver/tablet_service.cc
File src/kudu/tserver/tablet_service.cc:

http://gerrit.cloudera.org:8080/#/c/11428/11/src/kudu/tserver/tablet_service.cc@1875
PS11, Line 1875: Failing
s/Failing/Rejecting/



-- 
To view, visit http://gerrit.cloudera.org:8080/11428
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Idc0f77673e1f04a34ab1f5c1930bbaa2498b39bf
Gerrit-Change-Number: 11428
Gerrit-PatchSet: 11
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Tidy Bot
Gerrit-Comment-Date: Thu, 11 Oct 2018 20:49:09 +0000
Gerrit-HasComments: Yes