provider/aws: Implement RetentionPeriod Changes for Kinesis Stream

[stack72]

As requested in #5219

make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSKinesisStream' 2>~/tf.log
==> Checking that code complies with gofmt requirements...
GO15VENDOREXPERIMENT=1 go generate $(GO15VENDOREXPERIMENT=1 go list ./... | grep -v /vendor/)
TF_ACC=1 GO15VENDOREXPERIMENT=1 go test ./builtin/providers/aws -v -run=TestAccAWSKinesisStream -timeout 120m
=== RUN   TestAccAWSKinesisStream_basic
--- PASS: TestAccAWSKinesisStream_basic (93.62s)
=== RUN   TestAccAWSKinesisStream_retentionPeriod
--- PASS: TestAccAWSKinesisStream_retentionPeriod (179.95s)
PASS
ok      github.com/hashicorp/terraform/builtin/providers/aws    273.587s

[phinze]

Could rearrange this to make it a bit shorter and simpler:

if n == o {
  // log & return
}
if n > o {
  // increase
} else {
  // decrease
}
// waitforstate here, same for both

[phinze]

LGTM generally - one small suggestion inline

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.