You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@celeborn.apache.org by zh...@apache.org on 2023/01/31 08:30:54 UTC

[incubator-celeborn] branch branch-0.2 updated (5dc008fc -> ab224098)

This is an automated email from the ASF dual-hosted git repository.

zhouky pushed a change to branch branch-0.2
in repository https://gitbox.apache.org/repos/asf/incubator-celeborn.git


    from 5dc008fc [RELEASE] Bump 0.2.1-SNAPSHOT
     new 1b9ac1bc [RELEASE] Bump 0.2.0-incubating
     new ab224098 [CELEBORN-253][MINOR] Fix the wrongly resolve celeborn.ha.master.node.id issue if enable HA (#1188)

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 common/src/main/scala/org/apache/celeborn/common/CelebornConf.scala     | 1 +
 .../src/test/scala/org/apache/celeborn/common/CelebornConfSuite.scala   | 1 +
 pom.xml                                                                 | 2 +-
 3 files changed, 3 insertions(+), 1 deletion(-)


[incubator-celeborn] 01/02: [RELEASE] Bump 0.2.0-incubating

Posted by zh...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

zhouky pushed a commit to branch branch-0.2
in repository https://gitbox.apache.org/repos/asf/incubator-celeborn.git

commit 1b9ac1bcc3e138a0b82341c862157d032767cf6f
Author: Ethan Feng <et...@apache.org>
AuthorDate: Tue Jan 3 19:42:39 2023 +0800

    [RELEASE] Bump 0.2.0-incubating
---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 891b5560..086c51bb 100644
--- a/pom.xml
+++ b/pom.xml
@@ -52,7 +52,7 @@
   </distributionManagement>
 
   <properties>
-    <project.version>0.2.1-SNAPSHOT</project.version>
+    <project.version>0.2.0-incubating</project.version>
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
 


[incubator-celeborn] 02/02: [CELEBORN-253][MINOR] Fix the wrongly resolve celeborn.ha.master.node.id issue if enable HA (#1188)

Posted by zh...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

zhouky pushed a commit to branch branch-0.2
in repository https://gitbox.apache.org/repos/asf/incubator-celeborn.git

commit ab224098a22e7a43f2819da984de87ed1a71013d
Author: Rex(Hui) An <an...@126.com>
AuthorDate: Tue Jan 31 15:39:58 2023 +0800

    [CELEBORN-253][MINOR] Fix the wrongly resolve celeborn.ha.master.node.id issue if enable HA (#1188)
---
 common/src/main/scala/org/apache/celeborn/common/CelebornConf.scala      | 1 +
 common/src/test/scala/org/apache/celeborn/common/CelebornConfSuite.scala | 1 +
 2 files changed, 2 insertions(+)

diff --git a/common/src/main/scala/org/apache/celeborn/common/CelebornConf.scala b/common/src/main/scala/org/apache/celeborn/common/CelebornConf.scala
index 64b28106..31827e42 100644
--- a/common/src/main/scala/org/apache/celeborn/common/CelebornConf.scala
+++ b/common/src/main/scala/org/apache/celeborn/common/CelebornConf.scala
@@ -576,6 +576,7 @@ class CelebornConf(loadDefaults: Boolean) extends Cloneable with Logging with Se
     val nodeConfPrefix = extractPrefix(HA_MASTER_NODE_HOST.key, "<id>")
     getAllWithPrefix(nodeConfPrefix)
       .map(_._1)
+      .filterNot(_.equals("id"))
       .map(k => extractPrefix(k, "."))
       .distinct
   }
diff --git a/common/src/test/scala/org/apache/celeborn/common/CelebornConfSuite.scala b/common/src/test/scala/org/apache/celeborn/common/CelebornConfSuite.scala
index 1be8f13d..90d7bf34 100644
--- a/common/src/test/scala/org/apache/celeborn/common/CelebornConfSuite.scala
+++ b/common/src/test/scala/org/apache/celeborn/common/CelebornConfSuite.scala
@@ -152,6 +152,7 @@ class CelebornConfSuite extends RssFunSuite {
 
   test("extract masterNodeIds") {
     val conf = new CelebornConf()
+      .set("celeborn.ha.master.node.id", "1")
       .set("celeborn.ha.master.node.1.host", "clb-1")
       .set("celeborn.ha.master.node.2.host", "clb-1")
       .set("celeborn.ha.master.node.3.host", "clb-1")