do not remove spanner extraStatements in TFV (#6201) (#4442)

* do not remove extraStatements in validator

* update test instead of flagging TFV

* Revert "update test instead of flagging TFV"

This reverts commit 2bceda09eacad1be8563002c2fe515749caecf99.

* Added comment

* Update mmv1/templates/terraform/encoders/spanner_database.go.erb

Co-authored-by: Stephen Lewis (Burrows) <[email protected]>

Co-authored-by: Stephen Lewis (Burrows) <[email protected]>
Signed-off-by: Modular Magician <[email protected]>

Co-authored-by: Stephen Lewis (Burrows) <[email protected]>

Author: modular-magician

.changelog/6201.txt

@@ -0,0 +1,3 @@
+```release-note:none
+
+```

google-beta/resource_spanner_database.go

@@ -699,8 +699,9 @@ func resourceSpannerDatabaseEncoder(d *schema.ResourceData, meta interface{}, ob
 	log.Printf("[DEBUG] Preparing to create new Database. Any extra DDL statements will be applied to the Database in a separate API call")
 
 	delete(obj, "name")
-	delete(obj, "versionRetentionPeriod")
 	delete(obj, "instance")
+
+	delete(obj, "versionRetentionPeriod")
 	delete(obj, "extraStatements")
 	return obj, nil
 }