Leftover references to R9 in repository

[martincostello]

Description

The repo contains various references to the internal R9 project. At least some of these should be removed so that the open source repository is friendly to non-Microsoft contributors who are not familiar with that that is or what it means.

Here are all the references GitHub code search finds: r9 results

I noticed this because the issue template contains this:

Reproduction Steps

N/A

Expected behavior

Correct terminology is used.

Actual behavior

Internal code name is used.

Regression?

N/A

Known Workarounds

N/A

Configuration

N/A

Other information

No response

[danmoseley]

Thanks, yes much of this code came from an internal repo with that name. We should replace them with generic terms.

[danmoseley]

@RussKie @geeknoid some of these look like DI keys, event sources, counter names -- presumably fixed once shipped. Do we plan to ship these with R9 in the name? Can we avoid any R9 hits in the repo when we ship?

[geeknoid]

Yes, these are all telemetry things which Nikita will be taking care of soon.

[xakep139]

Yes, these are all telemetry things which Nikita will be taking care of soon.

Hopefully not only me 😀

[RussKie]

@xakep139 assigning to you for now. We all be cheering for you :)

[danmoseley]

Do we need to mark this somehow as "blocking release"/"necessary for GA"

[xakep139]

@xakep139 assigning to you for now. We all be cheering for you :)

I addressed some of these references in #4290

[geeknoid]

Still more references remain, so let's keep this open.

[xakep139]

@geeknoid, I think this one can be actually closed once #4482 is merged and #4554 is unblocked