Skip to content

Conversation

@Jam804
Copy link
Contributor

@Jam804 Jam804 commented Mar 14, 2025

Purpose of the pull request

fix the issue #4262
#4302
#4276

Fix Description

Issue: This PR addresses #4262, which arises due to the use of MyBatis logical deletion. MyBatis treats the is_deleted field as an int, whereas, in the PostgreSQL database, this field is of type boolean, causing a type incompatibility.

Changes Made
Field Type Modification:

Changed the data type of the is_deleted field in the PostgreSQL database from boolean to int to ensure compatibility with MyBatis.
Updated existing data to convert boolean values to int (e.g., true to 1 and false to 0).
After modifying the type, we also tested the MySQL database and found no issues.

image

image

aiwenmo and others added 2 commits March 8, 2025 16:42
…s with the same prefix cause errors in Flink configuration parsing (DataLinkDC#4286)

Co-authored-by: zhangyuhang <[email protected]>
Co-authored-by: yuhang2.zhang <[email protected]>
…s with the same prefix cause errors in Flink configuration parsing. (DataLinkDC#4298)

Co-authored-by: yuhang2.zhang <[email protected]>
@aiwenmo aiwenmo changed the base branch from 1.2.2 to 1.2 March 16, 2025 14:39
@Zzm0809
Copy link
Contributor

Zzm0809 commented Mar 21, 2025

删除字段的类型问题 可以参考 #3968 中的 dinky-admin/src/main/java/org/dinky/interceptor/PostgreSQLPrepareInterceptor.java 方式来处理

@Jam804
Copy link
Contributor Author

Jam804 commented Mar 21, 2025

删除字段的类型问题 可以参考 #3968 中的 dinky-admin/src/main/java/org/dinky/interceptor/PostgreSQLPrepareInterceptor.java 方式来处理

Doing it that way is troublesome; it requires modifying all the query SQLs, and if any are missed, it can easily lead to bugs. Are you sure you want to do it that way? @Zzm0809 @aiwenmo

@Jam804 Jam804 force-pushed the 1.2.2 branch 2 times, most recently from 91e9123 to 0fa4db5 Compare March 22, 2025 10:54
@Jam804 Jam804 closed this Mar 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants