Orleans 7 - PersistentStateAttribute - StateName really should be set

[zeus82]

I'm in the process of migrating from Orleans 3 to 7. With Orleans 3.x, Basically all my grains had something like this:

[PersistentState(null)] IPersistentState<State> data

Where the 'State' class was an inner class of my grain. This worked fine, but now with Orleans 7, these all somehow get data set as the GrainTypeString and so it seems like the ADO persistence provider gets confused because I see its trying to deserialise the wrong type. I was able to fix it by adding nameof(MyGrain) instead of null.

It would be great if now the PersistentStateAttribute prevented you from setting stateName to null!

[ReubenBond]

@zeus82 good idea