github-actions[bot] commented on code in PR #41019:
URL: https://github.com/apache/beam/pull/41019#discussion_r4483621901
##########
website/www/site/content/en/documentation/dsls/sql/extensions/create-external-table.md:
##########
@@ -574,12 +598,24 @@ The specified schema is used to read and write Parquet
files. The schema is conv
<td>string
</td>
</tr>
+ <tr>
+ <td>BINARY, VARBINARY
+ </td>
+ <td>bytes
+ </td>
+ </tr>
<tr>
<td>ARRAY
</td>
<td>array
</td>
</tr>
+ <tr>
+ <td>MAP
+ </td>
+ <td>map (keys must be VARCHAR, because Avro maps only support string keys)
Review Comment:
**🟢 Nit**: `CHAR` keys also map to `FieldType.STRING` in `CalciteUtils` and
are accepted by `AvroUtils` for Avro map keys, so you could optionally mention
`CHAR` or `VARCHAR` here.
```suggestion
<td>map (keys must be CHAR or VARCHAR, because Avro maps only support
string keys)
```
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
