Re: Discuss regarding Ticket 263

"Alex van den Bogaerdt" <[email protected]>
Newsgroups gmane.comp.db.rrdtool.devel
Message-ID <D6A18C916CFD4FBB97E990E7CADC5CCA@desk>
----- Original Message ----- 
From: "Jean-Edouard Babin" <[email protected]>
To: "Alex van den Bogaerdt" <[email protected]>
Cc: <[email protected]>
Sent: Monday, May 24, 2010 7:19 PM
Subject: Re: [rrd-developers] Discuss regarding Ticket 263



> Yes, conditional (G)PRINT would be better, but rrdtool set a "false"
> default time to the VDEF result anyway.

> > Using 0 was a design choice.

This is not a false timestamp, it is overloaded so to speak, 0 means false, 
unset.

> > I don't think time 0 will ever be valid, so instead of adding something 
> > to
> > this struct, why not simply if (vdef->when==0) printf("none"); else
> > {...whatever is done now...}

> Imagine someone doing statistic and importing data from 1970 to 2010,
> with one value per month and the fewest one on 1st Jan 1970.

I did back then, read on

> If he wants to print the first value and use a VDEF for that, he would
> not get it's result even if in his case 0 is the correct value.
> From my point of view adding 'never' flag is better as it can't break 
> something

Does RRDtool actually work properly when working with timestamps around 
1970?

When I programmed the (basics of) VDEF, it didn't.  If memory serves me 
well, it has been a while, so maybe the next changed, 1980-01-01 or so was 
the earliest date one could use (or reliably use).

What timestamp results from local time 1970-01-01 00:00 ?  It will result in 
0 for the UK, Iceland, and other places where during the winter there's no 
time difference with UTC. In a large part of Europe it will be 3600. In 
timezones west of Greenwich (UK) it will have to be a negative value.  Does 
that actually work? West of Greenwich includes the Americas, not really a 
small part of the world.

Bottom line:
If things have changed, perhaps 0 is no longer a suitable false value. But 
chances are it is as valid as it has been, so then use it.

HTH
Alex
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.