impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Youwei Wang (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-3504: UDF for current timestamp in UTC
Date Sat, 01 Oct 2016 01:50:14 GMT
Youwei Wang has posted comments on this change.

Change subject: IMPALA-3504: UDF for current timestamp in UTC

Patch Set 8:

File be/src/exprs/

PS6, Line 4120: cTime();
> forward-lapsing?
Greeting, Matthew.
As a commone sense, time never rewinds, time never goes back. Since I am not a native English
speaker, would you please share me a more elegant and native way to experss this?

Thank you for any idea or hint. :)

PS6, Line 4132: .UtcToLoc
> can you name these to reference the values they're holding, e.g. unixtime_u

PS6, Line 4136: EXPECT_TRUE(now
> This should be true unless there wasn't a value in the TimestampValue, whic

PS6, Line 4137: EXPECT_TRUE(std
> same
File be/src/exprs/

PS7, Line 4137: EXPECT
> missed this before, but this should be EXPECT_TRUE
File be/src/service/

PS6, Line 843: g(TimestampValue::LocalTime().DebugString());
> I'd still vote to not bother with the conversion, but it seems fine.
As you wish, Sir. :)

PS6, Line 845: 
             :   // Creating a random_generator every time is not free, but
> I'm not sure we should check this. While this seems intuitive, there might 

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I5ee6ee192aa469f77c711f27ad324696a42004d1
Gerrit-PatchSet: 8
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Youwei Wang <>
Gerrit-Reviewer: Jim Apple <>
Gerrit-Reviewer: Matthew Jacobs <>
Gerrit-Reviewer: Youwei Wang <>
Gerrit-HasComments: Yes

View raw message