<div dir="ltr"><div>How about we change the behavior and call it "expires_after".<br><br></div>--BDS<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 6, 2017 at 10:29 AM, Georg Fritzsche <span dir="ltr"><<a href="mailto:gfritzsche@mozilla.com" target="_blank">gfritzsche@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div>We seem to have this coming up repeatedly.<br></div>What naming would be more clear?<br></div><div>"recording_stops_in_version"?<br></div><div><br></div><div>Currently we use:<br></div><div>- Histograms.json uses "expires_in_version"<br></div>- Scalars.yaml uses "expires"<br></div>- Events.yaml uses "expiry_version"<br><br></div>We should end up using common field names and make them less confusing.<span class="HOEnZb"><font color="#888888"><br><br></font></span></div><span class="HOEnZb"><font color="#888888">Georg</font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 6, 2017 at 3:55 PM, Alessio Placitelli <span dir="ltr"><<a href="mailto:aplacitelli@mozilla.com" target="_blank">aplacitelli@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">It means that it stops being recorded in 55, see <a href="http://searchfox.org/mozilla-central/source/toolkit/components/telemetry/TelemetryCommon.cpp#28" target="_blank">http://searchfox.org/mozilla-c<wbr>entral/source/toolkit/componen<wbr>ts/telemetry/TelemetryCommon.<wbr>cpp#28</a><br></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="m_-5407494559260203231h5">2017-03-06 15:42 GMT+01:00 Benjamin Smedberg <span dir="ltr"><<a href="mailto:benjamin@smedbergs.us" target="_blank">benjamin@smedbergs.us</a>></span>:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="m_-5407494559260203231h5"><div dir="ltr"><div><div>When a histogram expires_in: 55, does that mean that it is recorded until Firefox 56, or that it stops being recorded in 55?<br><br></div>I always assumed that expires_in: 55 meant that it stopped being recorded in Firefox 56, but seeing some recent alert emails has made me less sure.<br><br></div>--BDS<br></div>
<br></div></div>______________________________<wbr>_________________<br>
fhr-dev mailing list<br>
<a href="mailto:fhr-dev@mozilla.org" target="_blank">fhr-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/fhr-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/listi<wbr>nfo/fhr-dev</a><br>
<br></blockquote></div><br></div>
<br>______________________________<wbr>_________________<br>
fhr-dev mailing list<br>
<a href="mailto:fhr-dev@mozilla.org" target="_blank">fhr-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/fhr-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/listi<wbr>nfo/fhr-dev</a><br>
<br></blockquote></div><br></div>
</div></div></blockquote></div><br></div>