Added new fields; removed deprecated fields; clarified recommended settings

[jeffreycs]

Three updates to the NRQL section:

• I've added the fields that were introduced with our new aggregation methods, which replace the deprecated evaluation_offset and since_value fields
• I've removed the deprecated fields from the example code
• I've adjusted the recommended values for aggregation_window and aggregation_delay to represent they're in seconds rather than minutes, addressing @HenryTech's comment here

I do have a question — by linking to their glossary definitions, is it clear that you need either aggregation_delay or aggregation_timer depending on which aggregation_method you've selected?

[CLAassistant]

All committers have signed the CLA.

[github-actions]

Hi @jeffreycs 👋
Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days.

[jeffreycs]

Well...... I've now realized that all the links to the glossary definitions are broken, likely due to a path change. I also appear to have missed an instance of since_value.

[gatsby-cloud]

Gatsby Cloud Build Report

docs-website-develop

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 15m

[paperclypse]

Thanks @jeffreycs! I'll look it over and see what I can do to get that sorted out.

[paperclypse]

Hi @jeffreycs, apparently we have an issue on our site when we link to pages through a redirect URL. It drops the ID. I've updated the URLs so we're using the direct URL.

Would you mind clarifying what you mean by "I also appear to have missed an instance of since_value."? Is there anything that I need to take care of there?

[jeffreycs]

Thanks @paperclypse! What I had meant is that I accidentally left one instance of a deprecated field. I just pushed a new commit that removes it.

[paperclypse]

Looks good!