WenDing-Y commented on PR #49: URL: https://github.com/apache/flink-connector-jdbc/pull/49#issuecomment-1577060660
> > Ok, I have rebase @MartijnVisser > > I don't think you have, given that there are 4 merge-commits in this PR which shouldn't be there when rebased properly: [bb13e1f](https://github.com/apache/flink-connector-jdbc/commit/bb13e1f51d071c5180ccf7d658839a3afac61468) [bced473](https://github.com/apache/flink-connector-jdbc/commit/bced473fbb8a606dca287fcf9329c7f052b9c4be) [9abdfaf](https://github.com/apache/flink-connector-jdbc/commit/9abdfafac76b17dcce371239ff5e16d9b90b9980) [2655b6b](https://github.com/apache/flink-connector-jdbc/commit/2655b6badefbea699ddb2465803076ec229b884b) > > What other tasks are needed for this PR to merge > > You will need to rebase, because there are a couple of merge commits when you merged `main` into your branch. Those merge commits prevent merging; by rebasing, those commits should not be needed anymore Sorry, it's my problem. I have rebase it again @MartijnVisser -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
