ECS debug names should be on by default

[seivan]

What problem does this solve or what need does it fill?

In order to save a few mb, we removed the things that should be on by default, even in release.
Error & crash logs, analytics, performance monitoring, all these things would require knowing the name of components, systems and etc.

What solution would you like?

I would not like to have another feature flag enabled just to get default convention for production enabled.
It should be the opposite, if you want to get rid of these names, have a flag for that.

I would like to see more focus on having things being default for something that will eventually run in production on peoples devices, as opposed to a bunch of hobby projects.
Removing things that would potentially be used for crash loggings, or analytics as default in order to save a few megabytes, seems counter-productive. You're going to have to turn it on at some point, no?

Am I missing something here?

[rparrett]

Linking #19558 for convenience. I think that's the change being referenced, anyhow.

[alice-i-cecile]

It should be the opposite, if you want to get rid of these names, have a flag for that.

Because of the way Rust feature flags work, this is not feasible. These are strictly additive: crates in your tree that need this information for some reason need to be able to count on "if I turned on the feature flag it will work". #21472 will make this a bit easier to work with at least.

I think it's reasonable to ship with these in production, but those space savings are very meaningful for some platforms (like web, embedded and mobile). But I agree with the existence of this feature flag and how it was implemented.