You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@inlong.apache.org by "featzhang (via GitHub)" <gi...@apache.org> on 2023/06/14 11:26:48 UTC
[GitHub] [inlong] featzhang opened a new pull request, #8243: [INLONG-8242][Tool] Separate the distribution module to improve the performance of CR pipeline
featzhang opened a new pull request, #8243:
URL: https://github.com/apache/inlong/pull/8243
The `inlong-distribution` module will cause parallel packaging failure and affect the performance of the CR pipeline, so this module is separated out.
When executing a CR pipeline, no packaging is required. Other modules can be compiled in parallel.
```shell
mvn clean package -B -T1C
```
Full package:
```shell
mvn clean package -Pdistribution
```
--
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@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [inlong] dockerzhang commented on pull request #8243: [INLONG-8242][CI] Separate the distribution module to improve the performance of check pipeline
Posted by "dockerzhang (via GitHub)" <gi...@apache.org>.
dockerzhang commented on PR #8243:
URL: https://github.com/apache/inlong/pull/8243#issuecomment-1598592628
@featzhang please update the workflow configurations, thanks.
--
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@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [INLONG-8242][CI] Separate the distribution module to improve the performance of check pipeline [inlong]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #8243:
URL: https://github.com/apache/inlong/pull/8243#issuecomment-2357336624
This PR is stale because it has been open for 60 days with no activity.
--
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@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [INLONG-8242][CI] Separate the distribution module to improve the performance of check pipeline [inlong]
Posted by "dockerzhang (via GitHub)" <gi...@apache.org>.
dockerzhang closed pull request #8243: [INLONG-8242][CI] Separate the distribution module to improve the performance of check pipeline
URL: https://github.com/apache/inlong/pull/8243
--
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@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [inlong] dockerzhang commented on a diff in pull request #8243: [INLONG-8242][CI] Separate the distribution module to improve the performance of check pipeline
Posted by "dockerzhang (via GitHub)" <gi...@apache.org>.
dockerzhang commented on code in PR #8243:
URL: https://github.com/apache/inlong/pull/8243#discussion_r1235128894
##########
pom.xml:
##########
@@ -1520,4 +1519,22 @@
<url>https://github.com/apache/inlong/actions</url>
</ciManagement>
+ <profiles>
+ <profile>
+ <id>distribution</id>
+ <activation>
+ <activeByDefault>true</activeByDefault>
+ </activation>
+ <modules>
+ <module>inlong-distribution</module>
+ </modules>
+ </profile>
+ <profile>
+ <id>fast</id>
Review Comment:
How about `fast` change to `parallel`? Here we should use a more accurate word.
--
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@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [inlong] healchow commented on a diff in pull request #8243: [INLONG-8242][Tool] Separate the distribution module to improve the performance of CR pipeline
Posted by "healchow (via GitHub)" <gi...@apache.org>.
healchow commented on code in PR #8243:
URL: https://github.com/apache/inlong/pull/8243#discussion_r1229466527
##########
pom.xml:
##########
@@ -48,7 +48,6 @@
<module>inlong-sort-standalone</module>
<module>inlong-manager</module>
<module>inlong-dashboard</module>
- <module>inlong-distribution</module>
Review Comment:
See https://github.com/apache/inlong/actions/runs/5266513756/workflow?#L91-#L110
--
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@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [inlong] healchow commented on a diff in pull request #8243: [INLONG-8242][Tool] Separate the distribution module to improve the performance of CR pipeline
Posted by "healchow (via GitHub)" <gi...@apache.org>.
healchow commented on code in PR #8243:
URL: https://github.com/apache/inlong/pull/8243#discussion_r1229463902
##########
pom.xml:
##########
@@ -48,7 +48,6 @@
<module>inlong-sort-standalone</module>
<module>inlong-manager</module>
<module>inlong-dashboard</module>
- <module>inlong-distribution</module>
Review Comment:
Please check out the GitHub action, we have an action to upload the package of distribution.
--
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@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [inlong] github-actions[bot] commented on pull request #8243: [INLONG-8242][CI] Separate the distribution module to improve the performance of check pipeline
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #8243:
URL: https://github.com/apache/inlong/pull/8243#issuecomment-1685496995
This PR is stale because it has been open for 60 days with no activity.
--
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@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org