[feature][connector-v2] add xugudb connector #6561
Conversation
L-Gryps
changed the title
TyrantLucifer
left a comment
TyrantLucifer
left a comment
There was a problem hiding this comment.
Thank you for your contribution. Please add CI trigger in https://github.com/apache/seatunnel/blob/dev/.github/workflows/backend.yml#L985 and rerun CI.
| case XUGU_VARCHAR: | ||
| case XUGU_VARCHAR2: | ||
| builder.dataType(BasicType.STRING_TYPE); | ||
| builder.columnLength(typeDefine.getLength()); |
There was a problem hiding this comment.
this length is char length or byte length ？
There was a problem hiding this comment.
Because multi-byte characters are handled differently in various databases, the byte length is uniformly used in seatunnel
use TypeDefineUtils.charTo4ByteLength(your length)
There was a problem hiding this comment.
There was a problem hiding this comment.
I have handled it in XuguTypeMapper
| @@ -0,0 +1,48 @@ | |||
| # | |||
There was a problem hiding this comment.
rename jdbc_xugu_source_and_upset_sink.conf to jdbc_xugu_source_and_upsert_sink.conf
|
|
||
| @Slf4j | ||
| @AutoService(TypeConverter.class) | ||
| public class XuguTypeConverter implements TypeConverter<BasicTypeDefine> { |
There was a problem hiding this comment.
|
|
||
| @Slf4j | ||
| @AutoService(TypeConverter.class) | ||
| public class XuguTypeConverter implements TypeConverter<BasicTypeDefine> { |
There was a problem hiding this comment.
| null, | ||
| null, | ||
| null, | ||
| null |
There was a problem hiding this comment.
yes，The Boolean data type has three states: true, false, and unknown. Field values can be true, false, null
5 participants
Purpose of this pull request
Does this PR introduce any user-facing change?
How was this patch tested?
Check list
New License Guide
release-note.