Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/10393 )
Change subject: IMPALA-6827: [DOCS] Updated the download link for the tutorial data ...................................................................... Patch Set 4: (3 comments) > I accidentally committed from a wrong branch and combined 2 > changes, here. > > Let me know if you want me to generate a html for kudu_tutorial for > review. I didn't see anything about kudu_tutorial here, just something about Kudu Tablet server replicas. Is that content relevant for both 3.x and 2.x? If it isn't, you need to separate out the commits, because the large change to impala_tutorial.xml can definitely go into 2.x http://gerrit.cloudera.org:8080/#/c/10393/4//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/10393/4//COMMIT_MSG@7 PS4, Line 7: IMPALA-6827: [DOCS] Updated the download link for the tutorial data Commit message has duplicate info. Please clean up the duplication. http://gerrit.cloudera.org:8080/#/c/10393/4/docs/topics/impala_tutorial.xml File docs/topics/impala_tutorial.xml: http://gerrit.cloudera.org:8080/#/c/10393/4/docs/topics/impala_tutorial.xml@1665 PS4, Line 1665: fi Was this the actual output or was it a mispaste? http://gerrit.cloudera.org:8080/#/c/10393/4/docs/topics/impala_tutorial.xml@1823 PS4, Line 1823: GROUOP Typo -- To view, visit http://gerrit.cloudera.org:8080/10393 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6823d1688169e0a6f09d5b552026bc18a3770828 Gerrit-Change-Number: 10393 Gerrit-PatchSet: 4 Gerrit-Owner: Alex Rodoni <[email protected]> Gerrit-Reviewer: Alex Rodoni <[email protected]> Gerrit-Reviewer: Harsh J <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jim Apple <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-Comment-Date: Thu, 17 May 2018 00:22:47 +0000 Gerrit-HasComments: Yes
