Skip to content

[cdc-common][pipeline-doris] Fix a typo and fix the doris pipeline connector.#2886

Merged
Jiabao-Sun merged 1 commit into
apache:masterfrom
joyCurry30:fix_time_type_precision
Dec 26, 2023
Merged

[cdc-common][pipeline-doris] Fix a typo and fix the doris pipeline connector.#2886
Jiabao-Sun merged 1 commit into
apache:masterfrom
joyCurry30:fix_time_type_precision

Conversation

@joyCurry30

Copy link
Copy Markdown
Contributor

Fix a typo of 'unsupported' and fix the doris pipeline connector using wrong value for 'precision' of time type.
The length of time type or timestamp type always be null.

image

cc @lvyanquan

@lvyanquan lvyanquan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@joyCurry30
joyCurry30 force-pushed the fix_time_type_precision branch from d44b92c to 32ce74d Compare December 19, 2023 04:49
…using wrong value for 'precision' of time type.
@joyCurry30
joyCurry30 force-pushed the fix_time_type_precision branch from 32ce74d to 2301def Compare December 21, 2023 11:23

@Jiabao-Sun Jiabao-Sun left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @joyCurry30 for this contribution.
LGTM.

@Jiabao-Sun
Jiabao-Sun merged commit 055b0af into apache:master Dec 26, 2023
lvyanquan pushed a commit that referenced this pull request Jan 18, 2024
joyCurry30 added a commit to joyCurry30/flink-cdc-connectors that referenced this pull request Mar 22, 2024
ChaomingZhangCN pushed a commit to ChaomingZhangCN/flink-cdc that referenced this pull request Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants