Skip to content
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.

Sign up for GitHub

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

Jump to bottom

proposal: time: add infinity/neg infinity time values as min/maximums for timestamps. #64525

Closed
james-lawrence opened this issue Dec 3, 2023 · 3 comments
Closed

proposal: time: add infinity/neg infinity time values as min/maximums for timestamps. #64525

james-lawrence opened this issue Dec 3, 2023 · 3 comments
Labels
Proposal WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.
Milestone
Proposal

Comments

@james-lawrence
Copy link
Contributor

Proposal Details

intended to be min/max values for timestamps. useful as signal values, defaults, etc. there is a lot to consider here. from serialization to semantics around time.Add, Since, etc, timezones, etc. if there is interest i'd be willing to write up a more formal proposal. mostly looking to gauge interest.
@gopherbot gopherbot added this to the Proposal milestone Dec 3, 2023
@seankhliao seankhliao added the WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. label Dec 3, 2023
@AlexanderYastrebov
Copy link
Contributor

Related issue #63844

@james-lawrence
Copy link
Contributor Author

somewhat related; inf/neg inf don't really care about the specific time; only that no time can be larger/smaller. i've found them incredibly useful for avoiding having a bunch of conditional logic in the code for where many people would use nil instead for various conditions.

@seankhliao seankhliao added WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. and removed WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. labels Dec 8, 2023
@gopherbot
Copy link

Timed out in state WaitingForInfo. Closing.

(I am just a bot, though. Please speak up if this is a mistake or you have the requested information.)

@gopherbot gopherbot closed this as not planned Won't fix, can't repro, duplicate, stale Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Proposal WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.
Projects
Proposals
Status: Incoming
Milestone
Proposal
Development

No branches or pull requests
4 participants
@AlexanderYastrebov @james-lawrence @gopherbot @seankhliao