Hi Sergey

Thanks for driving this FLIP, +1.

It may be a minor typo for the createTable method doc.

```

 * <pre>{@code
 * tEnv.createTemporaryTable("MyTable", TableDescriptor.forConnector("datagen")
 * .schema(Schema.newBuilder()
 * .column("f0", DataTypes.STRING())
 * .build())
 * .option(DataGenOptions.ROWS_PER_SECOND, 10)
 * .option("fields.f0.kind", "random")
 * .build());
 * }</pre>

```

Should `tEnv.createTemporaryTable` be `tEnv.createTable`?


Best
Ron

Sergey Nuyanzin <snuyan...@gmail.com> 于2024年12月18日周三 06:12写道:

> Hi all!
>
> I would like to open up for discussion another new small FLIP-494[1].
>
> Motivation
> This FLIP continues the journey of bringing Table API and FlinkSQL closer
> to each other.
> It adds missing create methods to TableEnvironment (alter functionality
> will be added in follow up FLIP(s)).
> There are already existing createTemporaryTable, createTable and
> createTemporaryView however still missing createView and createTable with
> ignoreIfExists flag
>
> [1]
> https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=334760466
>
>
> --
> Best regards,
> Sergey
>

Reply via email to