legendtkl commented on PR #1931: URL: https://github.com/apache/incubator-seatunnel/pull/1931#issuecomment-1133821152
> > Hi, @ruanwenjun and @BenJFan , I'm afraid the git workflow might need some improvement. > > I tried the failed command in my machine (MacOS), it works. > > I think the failure might caused by maven concurrency building '-T' > > ```shell > > ./mvnw -T 2C -B clean verify -Dmaven.test.skip=false -Dcheckstyle.skip=true -Dscalastyle.skip=true -Dlicense.skipAddThirdParty=true --no-snapshot-updates > > ``` > > > > I suggest to modify it with no concurrency thread build. > > What do you thinks? > > What do you seem to be missing? This CI will list the newly added dependencies, UT failed with this message "Error: Failed to execute goal org.apache.maven.plugins:maven-assembly-plugin:3.3.0:single (bin) on project seatunnel-dist: Execution bin of goal org.apache.maven.plugins:maven-assembly-plugin:3.3.0:single failed: basedir D:\a\incubator-seatunnel\incubator-seatunnel\seatunnel-dist\..\seatunnel-connectors\seatunnel-connectors-flink-sql-dist\target\lib does not exist -> [Help 1]" Just as I comment above, the maven concurrency build with `-T C2` might cause this problem. I run the same command on my laptop, it works. To verify this, I submit a new commit to remove the `-T C2` in git worklow, now the step in CI passed. So I think we need to update the Github CI workflow (already present in this PR) to avoid the misjudgement. Or other improvement of github CI works is ok. cc @CalvinKirs @ruanwenjun -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
