[GitHub] [incubator-seatunnel] ruanwenjun commented on pull request #1931: [Feature][Flink-SQL-connector] add flink sql connector hbase 1.2&2.2 and docs

[GitBox <gi...@apache.org>]

ruanwenjun commented on PR #1931:
URL: https://github.com/apache/incubator-seatunnel/pull/1931#issuecomment-1133798087

   > 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?
   
   Yeah, if you may get error if you use -T 2C to build in your laptop. But I don't think we need to remove the concurrency setting, since this is used for GitHub CI, you can just remove this when you build on your laptop.


-- 
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: commits-unsubscribe@seatunnel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org