[Fix][Seatunnel-api]support timestamp type

[litiliu]

Purpose of this pull request

This close #7125

Does this PR introduce any user-facing change?

How was this patch tested?

add UT case

Check list

• If any new Jar binary package adding in your PR, please add License Notice according
  New License Guide
• If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
• If you are contributing the connector code, please check that the following files are updated:
  1. Update change log that in connector document. For more details you can refer to connector-v2
  2. Update plugin-mapping.properties and add new connector information in it
  3. Update the pom file of seatunnel-dist
• Update the release-note.

[Hisoka-X]

In fact, SeaTunnel doesn't support Timestamp type at now. We should make sure flink layer not send timestamp type to seatunnel connector. Please refer

seatunnel/seatunnel-translation/seatunnel-translation-flink/seatunnel-translation-flink-common/src/main/java/org/apache/seatunnel/translation/flink/serialization/FlinkRowConverter.java

Line 45 in adafd80

public class FlinkRowConverter extends RowConverter<Row> {

cc @TyrantLucifer

[litiliu]

In fact, SeaTunnel doesn't support Timestamp type at now. We should make sure flink layer not send timestamp type to seatunnel connector. Please refer

seatunnel/seatunnel-translation/seatunnel-translation-flink/seatunnel-translation-flink-common/src/main/java/org/apache/seatunnel/translation/flink/serialization/FlinkRowConverter.java

Line 45 in adafd80

public class FlinkRowConverter extends RowConverter<Row> {

cc @TyrantLucifer

@Hisoka-X Seems the

seatunnel/seatunnel-api/src/main/java/org/apache/seatunnel/api/table/type/SeaTunnelRow.java

Line 157 in 8a48cee

case TIMESTAMP:

has supported the Timestamp type? Could you please explain why it's not supported?

[litiliu]

@hailin0 PTAL

[litiliu]

@liugddx PTAL

[TyrantLucifer]

@Hisoka-X In what scenarios will java.sql.Timestamp，I think it maybe connector has some problems. Under normal circumstances, it should be LocalDateTime.

[Hisoka-X]

@Hisoka-X In what scenarios will java.sql.Timestamp，I think it maybe connector has some problems. Under normal circumstances, it should be LocalDateTime.

@Hisoka-X In what scenarios will java.sql.Timestamp，I think it maybe connector has some problems. Under normal circumstances, it should be LocalDateTime.

Yep, you talk the point. After #7355, we removed FlinkRowConverter. So all data type came from connector without convert in flink.

[Hisoka-X]

@litiliu Please rebase on dev and check we still have same problem or not.