You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by "JiajunBernoulli (via GitHub)" <gi...@apache.org> on 2023/09/29 08:32:49 UTC

[GitHub] [calcite] JiajunBernoulli commented on a diff in pull request #3444: [MINOR] Add example for MAP type in reference docs

JiajunBernoulli commented on code in PR #3444:
URL: https://github.com/apache/calcite/pull/3444#discussion_r1341072293


##########
site/_docs/reference.md:
##########
@@ -1200,7 +1200,7 @@ Note:
 | ANY      | The union of all types |
 | UNKNOWN  | A value of an unknown type; used as a placeholder |
 | ROW      | Row with 1 or more columns | Example: Row(f0 int null, f1 varchar)
-| MAP      | Collection of keys mapped to values |
+| MAP      | Collection of keys mapped to values | Example: MAP\['washington', 1, 'obama', 44\]

Review Comment:
   This example is different from other examples.
   
   I think the example should be `map(varchar, int)`.



-- 
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: commits-unsubscribe@calcite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org