piotr-szuberski commented on a change in pull request #13319:
URL: https://github.com/apache/beam/pull/13319#discussion_r529421367
##########
File path:
website/www/site/content/en/documentation/dsls/sql/extensions/create-external-table.md
##########
@@ -204,6 +205,131 @@ TYPE bigquery
LOCATION 'testing-integration:apache.users'
```
+## Cloud Bigtable
+
+### Syntax
+
+```
+CREATE EXTERNAL TABLE [ IF NOT EXISTS ] tableName (
+ key VARCHAR NOT NULL,
+ family ROW<qualifier cells [, qualifier cells ]* >
+ [, family ROW< qualifier cells [, qualifier cells ]* > ]*
+)
+TYPE bigtable
+LOCATION
'googleapis.com/bigtable/projects/[PROJECT_ID]/instances/[INSTANCE_ID]/tables/[TABLE]'
+```
+
+* `key`: key of the Bigtable row
+* `family`: name of the column family
+* `qualifier`: the column qualifier
+* `cells`: Either of each value:
+ * `TYPE`
+ * `ARRAY<SIMPLE_TYPE>`
+* `LOCATION`:
+ * `PROJECT_ID`: ID of the Google Cloud Project.
+ * `INSTANCE_ID`: Bigtable instance ID.
+ * `TABLE`: Bigtable Table ID.
+* `TYPE`: `SIMPLE_TYPE` or `CELL_ROW`
+* `CELL_ROW`: `ROW<val SIMPLE_TYPE [, timestampMicros BIGINT [NOT NULL]] [,
labels ARRAY<VARCHAR> [NOT NULL]]`
+* `SIMPLE_TYPE`: on of the following:
+ * `BINARY`
+ * `VARCHAR`
+ * `BIGINT`
+ * `DOUBLE`
+ * `BOOLEAN`
Review comment:
BigQuery supports only those 5 types. But I agree that there is no
reason why not to support others as well. Should I do that in this PR or in the
next one?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
