You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@celeborn.apache.org by et...@apache.org on 2025/01/03 06:33:10 UTC
(celeborn) 01/03: [CELEBORN-1240][FOLLOWUP] Introduce web profile for web module
This is an automated email from the ASF dual-hosted git repository.
ethanfeng pushed a commit to branch branch-0.5
in repository https://gitbox.apache.org/repos/asf/celeborn.git
commit 9944b20246841e69f8ecbd36096b18d38c603568
Author: SteNicholas <pr...@163.com>
AuthorDate: Tue Aug 27 15:00:52 2024 +0800
[CELEBORN-1240][FOLLOWUP] Introduce web profile for web module
### What changes were proposed in this pull request?
Introduce web profile for web module.
### Why are the changes needed?
The compilation speed of web module is sometimes very slow due to the influence of the network, which hinders the development process.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
CI.
Closes #2679 from SteNicholas/CELEBORN-1240.
Authored-by: SteNicholas <pr...@163.com>
Signed-off-by: mingji <fe...@alibaba-inc.com>
(cherry picked from commit c0dda4a15aa988ba1dca9437573bb8aed8c1554c)
---
pom.xml | 8 +++++++-
project/CelebornBuild.scala | 11 ++++++++++-
2 files changed, 17 insertions(+), 2 deletions(-)
diff --git a/pom.xml b/pom.xml
index febfb5813..8858f0e35 100644
--- a/pom.xml
+++ b/pom.xml
@@ -36,7 +36,6 @@
<module>service</module>
<module>master</module>
<module>worker</module>
- <module>web</module>
</modules>
<distributionManagement>
@@ -1634,6 +1633,13 @@
</modules>
</profile>
+ <profile>
+ <id>web</id>
+ <modules>
+ <module>web</module>
+ </modules>
+ </profile>
+
<profile>
<id>google-mirror</id>
<properties>
diff --git a/project/CelebornBuild.scala b/project/CelebornBuild.scala
index f5fe2e085..692dc3ee9 100644
--- a/project/CelebornBuild.scala
+++ b/project/CelebornBuild.scala
@@ -336,7 +336,7 @@ object CelebornBuild extends sbt.internal.BuildDef {
CelebornClient.client,
CelebornService.service,
CelebornWorker.worker,
- CelebornMaster.master) ++ maybeSparkClientModules ++ maybeFlinkClientModules ++ maybeMRClientModules
+ CelebornMaster.master) ++ maybeSparkClientModules ++ maybeFlinkClientModules ++ maybeMRClientModules ++ maybeWebModules
}
// ThisBuild / parallelExecution := false
@@ -401,6 +401,15 @@ object Utils {
lazy val maybeMRClientModules: Seq[Project] = mrClientProjects.map(_.modules).getOrElse(Seq.empty)
+ val WEB_VERSION = profiles.filter(_.startsWith("web")).headOption
+
+ lazy val webProjects = WEB_VERSION match {
+ case Some("web") => Some(WebProjects)
+ case _ => None
+ }
+
+ lazy val maybeWebModules: Seq[Project] = webProjects.map(_.modules).getOrElse(Seq.empty)
+
def defaultScalaVersion(): String = {
// 1. Inherit the scala version of the spark project
// 2. if the spark profile not specified, using the DEFAULT_SCALA_VERSION