jonathansudhakar1 commented on code in PR #41019:
URL: https://github.com/apache/beam/pull/41019#discussion_r4495291768
##########
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:
Thanks, done in f376eab122e. The cell now says "keys must be CHAR or
VARCHAR". `CalciteUtils` maps both types to `FieldType.STRING`. I also checked
it with a temporary JUnit test (not committed) in `:sdks:java:extensions:sql`:
a `MAP<CHAR, INTEGER>` column converts to Avro
`["null",{"type":"map","values":"int"}]`, and `INSERT ... MAP['k', 9]` followed
by `SELECT *` reads back `{k=9}` on the DirectRunner.
--
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]
