-
Notifications
You must be signed in to change notification settings - Fork 153
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(date): add location support to date.truncate function #4675
feat(date): add location support to date.truncate function #4675
Conversation
CC @skartikey |
5cf4ff7
to
5c3affa
Compare
if err != nil { | ||
return nil, err | ||
} | ||
b := w.GetLatestBounds(t) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Strange how we have two "Window" types and one supports GetEarliestBounds
and the other only supports GetLatestBounds
. Since the windows do not overlap they ought to return the same result either way at least (nothing to change here).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed! BTW execute.Window
seems to only be used by interpolate
. So I think may be able to get rid of it and completely replace it by interval.Window
.
@Marwes PTAL! |
@Marwes friendly ping |
execute.Window
byinterval.Window
Inspired by #4327
fixes #4656
Done checklist