-
Notifications
You must be signed in to change notification settings - Fork 2k
[docs][minor] Correct Typos in Documentation/Code #3451
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
@yuxiqian PTAL |
Thanks for @Karl-WangSK's contribution! It reminds me that JetBrains IDE always complains about typos in code comments. Would you like to fix them together if possible? |
...k-cdc-common/src/main/java/org/apache/flink/cdc/common/configuration/ConfigurationUtils.java
Outdated
Show resolved
Hide resolved
Looks good! Since some changes affected test cases, could @ruanhang1993 please trigger the CI? |
hello , pls can trigger the CI e2e check? this cancelled not related to this pr. @ruanhang1993 @leonardBang |
Seems
|
ready to merge @leonardBang |
...e/src/main/java/org/apache/flink/cdc/runtime/serializer/schema/MetadataColumnSerializer.java
Outdated
Show resolved
Hide resolved
Hi @Karl-WangSK, any updates on this PR? |
sry for late, revised |
This pull request has been automatically marked as stale because it has not had recent activity for 60 days. It will be closed in 30 days if no further activity occurs. |
@Karl-WangSK Would you like to rebase this PR? and then we can review this PR again. |
This change ensures that the log message is clear and free of typographical errors.