Skip to content

[FLINK-37203] Alter table comment or column comment #3893

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.

Sign up for GitHub

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

Jump to bottom
Open
ChaomingZhangCN wants to merge 6 commits into
base: master
Choose a base branch
from
Open

[FLINK-37203] Alter table comment or column comment #3893

ChaomingZhangCN wants to merge 6 commits into from

Conversation

@ChaomingZhangCN
Copy link
Contributor

@ChaomingZhangCN ChaomingZhangCN commented Jan 24, 2025

@ChaomingZhangCN
Copy link
Contributor Author

ChaomingZhangCN commented Jan 24, 2025

@lvyanquan @leonardBang PTAL.

@ChaomingZhangCN
Copy link
Contributor Author

ChaomingZhangCN commented Mar 21, 2025

@yuxiqian Can you help review?

@lvyanquan lvyanquan self-assigned this May 27, 2025
@github-actions
Copy link

github-actions bot commented Sep 25, 2025

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

@github-actions github-actions bot added the Stale label Sep 25, 2025
@lvyanquan lvyanquan removed the Stale label Sep 25, 2025
@lvyanquan
Copy link
Contributor

lvyanquan commented Dec 3, 2025

Hi @ChaomingZhangCN, I apologize for not reviewing your changes in time during the past period due to other commitments. I believe this modification is reasonable. You can rebase with master (there may be conflicts), and I will review it in the coming days.

@leonardBang leonardBang reopened this Dec 3, 2025
@ChaomingZhangCN
Copy link
Contributor Author

ChaomingZhangCN commented Dec 10, 2025

Hi @ChaomingZhangCN, I apologize for not reviewing your changes in time during the past period due to other commitments. I believe this modification is reasonable. You can rebase with master (there may be conflicts), and I will review it in the coming days.

No problem at all, thank you for the update! I appreciate your trust in the changes. I'll rebase with master and address any conflicts. Looking forward to your feedback in the coming days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lvyanquan lvyanquan Awaiting requested review from lvyanquan

At least 1 approving review is required to merge this pull request.

Assignees

@lvyanquan lvyanquan

Labels

common composer doris-pipeline-connector e2e-tests iceberg-pipeline-connector maxcompute-pipeline-connector mysql-pipeline-connector oceanbase-pipeline-connector paimon-pipeline-connector runtime starrocks-pipeline-connector values-pipeline-connector

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ChaomingZhangCN @lvyanquan @leonardBang