twalthr commented on a change in pull request #14594:
URL: https://github.com/apache/flink/pull/14594#discussion_r559393915
##########
File path: docs/dev/table/types.md
##########
@@ -152,6 +152,42 @@ a table program (e.g.
`field.cast(TIMESTAMP(3).bridgedTo(Timestamp.class))`) are
</div>
</div>
+<div class="codetabs" data-hide-tabs="1" markdown="1">
+<div data-lang="Java/Scala" markdown="1">
+The default planner supports the following set of SQL types:
+
+| Data Type | Remarks for Data Type |
+|:----------|:----------------------|
+| `CHAR` | |
+| `VARCHAR` | |
+| `STRING` | |
+| `BOOLEAN` | |
+| `BYTES` | `BINARY` and `VARBINARY` are not supported yet. |
+| `DECIMAL` | Supports fixed precision and scale. |
+| `TINYINT` | |
+| `SMALLINT` | |
+| `INTEGER` | |
+| `BIGINT` | |
+| `FLOAT` | |
+| `DOUBLE` | |
+| `DATE` | |
+| `TIME` | Supports only a precision of `0`. |
+| `TIMESTAMP` | |
+| `TIMESTAMP WITH LOCAL TIME ZONE` | |
+| `INTERVAL` | Supports only interval of `MONTH` and `SECOND(3)`. |
+| `ARRAY` | |
+| `MULTISET` | |
+| `MAP` | |
+| `ROW` | |
+| `RAW` | |
+| structured types | Only exposed in user-defined functions yet. |
+
+</div>
+<div data-lang="Python" markdown="1">
+N/A
Review comment:
not very nice, but I guess we need a separate issue for this
##########
File path: docs/dev/table/common.md
##########
@@ -122,7 +122,7 @@ The `TableEnvironment` is the entrypoint for Table API and
SQL integration and i
* Executing SQL queries
* Registering a user-defined (scalar, table, or aggregation) function
* Converting a `DataStream` into a `Table`
-* Holding a reference to a `StreamExecutionEnvironment`
+* In case of StreamTableEnvironment holding a reference to a
`StreamExecutionEnvironment`
Review comment:
nit: I would surround it with back ticks
----------------------------------------------------------------
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:
[email protected]
