David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1189 scans: reuse snapshot timestamp when not set ......................................................................
Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/5143/5//COMMIT_MSG Commit Message: PS5, Line 7: scans: reuse snapshot timestamp when not set > Too long: far more than 50 symbols :) We follow that guideline at the authors discretion if you look at the commit log. Commit message titles are important in the sense that they are the only things that appear on a condensed log view. That being said it was more about rephasing that anything else. How about: Reuse snapshot scan timestamp across tablets -- To view, visit http://gerrit.cloudera.org:8080/5143 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2ac708e38b8a80834f7d54eca294517cbfb06ec6 Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-HasComments: Yes