sjwiesman commented on a change in pull request #10078: [FLINK-14486][table-api, docs] Update documentation regarding Temporary Objects URL: https://github.com/apache/flink/pull/10078#discussion_r342227368
########## File path: docs/dev/table/common.md ########## @@ -345,133 +359,139 @@ tableEnv.registerTable("projectedTable", projTable) table_env = ... # see "Create a TableEnvironment" section # table is the result of a simple projection query -proj_table = table_env.scan("X").select(...) +proj_table = table_env.from_path("X").select(...) # register the Table projTable as table "projectedTable" table_env.register_table("projectedTable", proj_table) {% endhighlight %} </div> </div> -**Note:** A registered `Table` is treated similarly to a `VIEW` as known from relational database systems, i.e., the query that defines the `Table` is not optimized but will be inlined when another query references the registered `Table`. If multiple queries reference the same registered `Table`, it will be inlined for each referencing query and executed multiple times, i.e., the result of the registered `Table` will *not* be shared. +**Note:** A `Table` object similarly to a `VIEW` from relational database Review comment: ```suggestion **Note:** `Table` objects are similar to `VIEW`'s from relational database ``` ---------------------------------------------------------------- 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 With regards, Apache Git Services