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.

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

Why is there a difference between the $INF and the infinity the temporal functions insert #3

Open
peetkes opened this issue Apr 21, 2016 · 2 comments

Comments

@peetkes
Copy link

peetkes commented Apr 21, 2016

I noticed that when we set the infinity for valEnd we use 9999-12-31T23:59:59.99Z and what we get when we put a null-node{} into the sysEnd we get 9999-12-31T11:59:59.99Z
Is that for a particular reason?

@fxue
Copy link
Owner

fxue commented Apr 21, 2016

Hi Peter,

You should use 9999-12-31T11:59:59.99Z because the other one will overflow
in some time zones.
The example are a bit outdated, they should all use 11am instead of 11pm.

On Thu, Apr 21, 2016 at 1:51 PM, Peter Kester [email protected]
wrote:

I noticed that when we set the infinity for valEnd we use
9999-12-31T23:59:59.99Z and what we get when we put a null-node{} into the
sysEnd we get 9999-12-31T11:59:59.99Z
Is that for a particular reason?


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#3

@peetkes
Copy link
Author

peetkes commented Apr 22, 2016

Thanks Fei. Will change it accordingly.

On 21 apr. 2016, at 22:12, Fei Xue [email protected] wrote:

Hi Peter,

You should use 9999-12-31T11:59:59.99Z because the other one will overflow
in some time zones.
The example are a bit outdated, they should all use 11am instead of 11pm.

On Thu, Apr 21, 2016 at 1:51 PM, Peter Kester [email protected]
wrote:

I noticed that when we set the infinity for valEnd we use
9999-12-31T23:59:59.99Z and what we get when we put a null-node{} into the
sysEnd we get 9999-12-31T11:59:59.99Z
Is that for a particular reason?


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#3


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants