Fixed non-unique ids in redshift_grant possible

[rg00d]

Currently, there is a risk of obtaining redshift_grant resources having identical id. This fix allows to avoid it.

[mtesch-um]

Just a comment - probably it would be sufficient to simply use - as a separator instead of _ because neither schema nor group names can contain that character. I think that would also make a pass-through import format for the resource simpler?

I wonder, does this need a state migration to the new state format? https://www.terraform.io/plugin/sdkv2/resources/state-migration

[rg00d]

Just a comment - probably it would be sufficient to simply use - as a separator instead of _ because neither schema nor group names can contain that character. I think that would also make a pass-through import format for the resource simpler?

We have checked this possibility and it seems that discussed vulnerability has still existed.

I wonder, does this need a state migration to the new state format? https://www.terraform.io/plugin/sdkv2/resources/state-migration

We have decided not to incorporate a state upgrader since this provider is still in the initial development phase.
ID will be recreated as a result of each Update action since it utilises the same underlying resourceRedshiftGrantCreate() function.