velox
velox copied to clipboard
Add Presto function at_timezone
Implement at time zone feature from Presto Java, for TimestampWithTimezone input.
Example usage:
SELECT timestamp '2012-10-31 01:00 UTC' AT TIME ZONE 'America/Los_Angeles';
2012-10-30 18:00:00.000 America/Los_Angeles
Deploy Preview for meta-velox canceled.
| Name | Link |
|---|---|
| Latest commit | 642b7f7489505359267449940a1f47b1d71442d9 |
| Latest deploy log | https://app.netlify.com/sites/meta-velox/deploys/6647d8444cfe0900089db02c |
Rebased and modified to use new type implementation.
@pedroerp would appreciate if you could take a look!
@pedroerp has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@svm1 looks like a bunch of builds are failing.
Rebased as my branch was a bit out of date, should take care of the build issues.
@pedroerp no worries, thanks for all your feedback!
Fixed the build errors, had to update my initialize() method - and added docs as I realized I'd forgotten that.
@pedroerp has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@pedroerp merged this pull request in facebookincubator/velox@8239bbdc67e05c9bd7430c127f958775dfb25405.
Conbench analyzed the 1 benchmark run on commit 8239bbdc.
There were no benchmark performance regressions. 🎉
The full Conbench report has more details.