Remove the feature flag and change the default for open generated files

[jasonmalinowski]

[jasonmalinowski]

Right now creating this to ensure no functional/performance regressions in divisional tests before we flip the feature flag. This will merge at some later point when we're comfortable with it.

[JoeRobich]

Your PR Validation run is out of date and contained failures.

[jasonmalinowski]

Internal validation run passed, so this is ready for review.

[CyrusNajmabadi]

This def worries me. But i guess we have feature flag to roll back if this causes an issue?

[CyrusNajmabadi]

Ok. So i'm fine flipping this on by default. But i'd say we should still have an available flag to disable thsi remotely if it turns out to be an issue.

[jasonmalinowski]

@CyrusNajmabadi Actually in that case then I can just flip the existing feature flag on, and we can just merge this in 17.6.

[CyrusNajmabadi]

That works for me.

[jasonmalinowski]

This is being merged into 17.6, but the existing feature flag is being flipped for 17.5.

[Cosifne]

Target main since it is 17.6 now

[jasonmalinowski]

@CyrusNajmabadi So we've had this feature flag rolled out to 100% of users for awhile with few reported issues so I'd like to merge this for 17.7; I don't see any reason to keep this feature flag around at this point.

[CyrusNajmabadi]

Works for me!

[jasonmalinowski]

@CyrusNajmabadi Ready for review then after resolving merge conflicts.