[GitHub] [flink-connector-elasticsearch] JingGe commented on pull request #2: [Flink-26884][draft] move elasticsearch connectors to the external repo

JingGe commented on PR #2: URL: https://github.com/apache/flink-connector-elasticsearch/pull/2#issuecomment-1087480666 please refer to the new PR: #5 -- 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 g

[GitHub] [flink-connector-elasticsearch] JingGe commented on pull request #2: [Flink-26884][draft] move elasticsearch connectors to the external repo

JingGe commented on pull request #2: URL: https://github.com/apache/flink-connector-elasticsearch/pull/2#issuecomment-1085750093 Thanks @MartijnVisser for the feedback. Like we talked offline, I will closed this PR and open a follow-up PR which will contain the fix of the dependency conve

[GitHub] [flink-connector-elasticsearch] JingGe commented on pull request #2: [Flink-26884][draft] move elasticsearch connectors to the external repo

JingGe commented on pull request #2: URL: https://github.com/apache/flink-connector-elasticsearch/pull/2#issuecomment-1084815636 > @JingGe I was testing a bit with a build pipeline and ran a convergence check. That actually fails. I ran `mvn clean install -Dflink.convergence.phase=install