Austin Nobis has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13293 )

Change subject: IMPALA-8497: dealing with query ends with '\n'
......................................................................


Patch Set 4:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/13293/4//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/13293/4//COMMIT_MSG@10
PS4, Line 10: Syntax
nit: syntax*


http://gerrit.cloudera.org:8080/#/c/13293/4//COMMIT_MSG@11
PS4, Line 11: sumbit
nit: submit*


http://gerrit.cloudera.org:8080/#/c/13293/4//COMMIT_MSG@15
PS4, Line 15: Tests:
            :   * Add test cases in ParserTest.java
Did you run the FE tests via maven? If so, add it here. If not, please run the 
FE tests.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3f034b351d0468a77773f6482e27ddef818b34d8
Gerrit-Change-Number: 13293
Gerrit-PatchSet: 4
Gerrit-Owner: wangsheng <[email protected]>
Gerrit-Reviewer: Austin Nobis <[email protected]>
Gerrit-Reviewer: Fredy Wijaya <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Comment-Date: Mon, 13 May 2019 16:57:41 +0000
Gerrit-HasComments: Yes

Reply via email to