Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10861 )
Change subject: IMPALA-3956: [DOCS] Escape variables with '\' in impala-shell ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/10861/1/docs/topics/impala_shell_running_commands.xml File docs/topics/impala_shell_running_commands.xml: http://gerrit.cloudera.org:8080/#/c/10861/1/docs/topics/impala_shell_running_commands.xml@72 PS1, Line 72: Use <codeph>--</codeph> or <codeph>/* */</codeph> to denote a comment. Use -- to denote a single comment or /* */ to denote a multiline comment. http://gerrit.cloudera.org:8080/#/c/10861/1/docs/topics/impala_shell_running_commands.xml@84 PS1, Line 84: $ It should be ${variable_name} -- To view, visit http://gerrit.cloudera.org:8080/10861 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifb95785a143939a94d55d3565364afe1e26c1f3d Gerrit-Change-Number: 10861 Gerrit-PatchSet: 1 Gerrit-Owner: Alex Rodoni <arod...@cloudera.com> Gerrit-Reviewer: Adam Holley <ahol...@cloudera.com> Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Comment-Date: Wed, 04 Jul 2018 06:53:11 +0000 Gerrit-HasComments: Yes