John Russell has posted comments on this change. Change subject: Updates to DML statements for Impala + Kudu ......................................................................
Patch Set 1: (2 comments) All comments addressed. I suggest we expedite the +2'ing of this one so we can get a clear picture in master of the various places touched by Kudu, and then if we want to add more details or examples, we'll follow up with a separate code review. http://gerrit.cloudera.org:8080/#/c/5646/1//COMMIT_MSG Commit Message: PS1, Line 7: Updates to DML statements for Impala + Kudu > FYI: I posted this PDF: Done http://gerrit.cloudera.org:8080/#/c/5646/1/docs/topics/impala_update.xml File docs/topics/impala_update.xml: Line 50: [ WHERE <varname>where_conditions</varname> ] > looks like the syntax is: Done -- To view, visit http://gerrit.cloudera.org:8080/5646 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I60512b7957fb53d86d3123a4f1d46fbb355f4665 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: John Russell <[email protected]> Gerrit-Reviewer: Ambreen Kazi <[email protected]> Gerrit-Reviewer: Dimitris Tsirogiannis <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: John Russell <[email protected]> Gerrit-Reviewer: Matthew Jacobs <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
