[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.

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity for 120 days. It will be closed in 7 days if no further activity occurs.

[github-actions]

This pull request has been closed because it has not had recent activity. You could reopen it if you try to continue your work, and anyone who are interested in it are encouraged to continue work on this pull request.