adding attribute family for bigtable_table

[eraac]

Fix #1576

make testacc TESTARGS='-run=TestAccBigtableTable'

==> Checking that code complies with gofmt requirements...
TF_ACC=1 TF_SCHEMA_PANIC_ON_ERROR=1 go test $(go list ./... |grep -v 'vendor') -v -run=TestAccBigtableTable -timeout 120m
?   	github.com/terraform-providers/terraform-provider-google	[no test files]
=== RUN   TestAccBigtableTable_basic
=== PAUSE TestAccBigtableTable_basic
=== RUN   TestAccBigtableTable_splitKeys
=== PAUSE TestAccBigtableTable_splitKeys
=== RUN   TestAccBigtableTable_family
=== PAUSE TestAccBigtableTable_family
=== CONT  TestAccBigtableTable_basic
=== CONT  TestAccBigtableTable_family
=== CONT  TestAccBigtableTable_splitKeys
--- PASS: TestAccBigtableTable_splitKeys (16.90s)
--- PASS: TestAccBigtableTable_basic (17.84s)
--- PASS: TestAccBigtableTable_family (19.11s)
PASS
ok  	github.com/terraform-providers/terraform-provider-google/google	19.154s
testing: warning: no tests to run
PASS
ok  	github.com/terraform-providers/terraform-provider-google/scripts	(cached) [no tests to run]

[rileykarson]

Great work! I have an ask about how we represent the field in schema - I think we might want to support an additional field in the future and we can save ourselves from a breaking change later by shaping the schema a little differently now.

[eraac]

@rileykarson I have push new code - updating code, documentation and test. Thanks for the feedback!

Not 100% sure about the code, waiting for a review

[rileykarson]

Looks great! Just a couple more small comments.

[rileykarson]

Thanks for contributing!

[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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!