Youwei Wang has posted comments on this change.

Change subject: IMPALA-2459: Implement next_day date/time UDF
......................................................................


Patch Set 9:

Hi Marcel. 
I am not sure what has happened. But if I open the file 
be/src/exprs/timestamp-functions.cc of PatchSet 5, your comment doesn't show up 
and I can't respond to it in this case.

"http://gerrit.cloudera.org:8080/#/c/1943/5/be/src/exprs/timestamp-functions-ir.cc
File be/src/exprs/timestamp-functions-ir.cc:
Line 582
> that doesn't mean you should remove all blank lines. please put them back.
please respond."

I don't know whether it is a bug of Gerrit or not. And I am sorry for any 
inconvenience I have caused due to this.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2721d236c096639a9e7d2df8a45ca888c6b3e83e
Gerrit-PatchSet: 9
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Youwei Wang <[email protected]>
Gerrit-Reviewer: Henry Robinson <[email protected]>
Gerrit-Reviewer: Jim Apple <[email protected]>
Gerrit-Reviewer: Lars Volker <[email protected]>
Gerrit-Reviewer: Marcel Kornacker <[email protected]>
Gerrit-Reviewer: Youwei Wang <[email protected]>
Gerrit-HasComments: No

Reply via email to