-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[pkg/ottl] New Unix
converter to convert epoch timestamp to time.Time
#27868
Comments
Unix
converter to convert epoch timestamp to time.TimeUnix
converter to convert epoch timestamp to time.Time
I'll take this one. |
All yours, thanks @kentquirk! |
I think this issue can be closed since it was solved as part of #29088 |
@iblancasa this issue has not yet been resolved. #29088 was closed since it is a duplicate of this issue. |
Oh. Sorry. I can work on it |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
I'll apply the changes requested in the PR today |
Solves #27868. --------- Signed-off-by: Israel Blancas <[email protected]> Co-authored-by: Tyler Helmuth <[email protected]>
Closed via #31556 |
Component(s)
pkg/ottl
Describe the issue you're reporting
[### Component(s)
pkg/ottl
Is your feature request related to a problem? Please describe.
OTTL has no way to create a
time.Time
from an epoch time.Describe the solution you'd like
A new
Unix
converter function. similar to Golang's Unix function, that converts the integer input to atime.Time
.The function should have 2 parameters. The first parameter should be the seconds input and is a
IntGetter
. The second input should be an optionalIntGetter
and represents the nanoseconds.The implementing PR should
pkg/ottl/ottlfuncs/functions.go
pkg/ottl/ottlfuncs/README.md
The text was updated successfully, but these errors were encountered: