[incubator-uniffle] branch master updated: Change url of total lines badge in README (#222)

2022-09-15 Thread jshao
This is an automated email from the ASF dual-hosted git repository.

jshao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-uniffle.git


The following commit(s) were added to refs/heads/master by this push:
 new 8be68ab4 Change url of total lines badge in README (#222)
8be68ab4 is described below

commit 8be68ab42de921e36073024c9bd2f08ae4814b23
Author: Kaijie Chen 
AuthorDate: Thu Sep 15 18:49:06 2022 +0800

Change url of total lines badge in README (#222)
---
 README.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 8794e016..4ab67422 100644
--- a/README.md
+++ b/README.md
@@ -24,7 +24,7 @@ Currently it supports [Apache 
Spark](https://spark.apache.org) and [MapReduce](h
 
 
[![Build](https://github.com/apache/incubator-uniffle/actions/workflows/build.yml/badge.svg?branch=master=push)](https://github.com/apache/incubator-uniffle/actions/workflows/build.yml)
 
[![Codecov](https://codecov.io/gh/apache/incubator-uniffle/branch/master/graph/badge.svg)](https://codecov.io/gh/apache/incubator-uniffle)
-[![Total 
Lines](https://img.shields.io/tokei/lines/github/apache/incubator-uniffle)](https://github.com/apache/incubator-uniffle)
+[![](https://sloc.xyz/github/apache/incubator-uniffle)](https://github.com/apache/incubator-uniffle)
 [![Code 
Quality](https://img.shields.io/lgtm/grade/java/github/apache/incubator-uniffle?label=code%20quality)](https://lgtm.com/projects/g/apache/incubator-uniffle/)
 
[![License](https://img.shields.io/github/license/apache/incubator-uniffle)](https://github.com/apache/incubator-uniffle/blob/master/LICENSE)
 



[incubator-uniffle] branch branch-0.6 updated: Fix incorrect log format strings (#220)

2022-09-15 Thread roryqi
This is an automated email from the ASF dual-hosted git repository.

roryqi pushed a commit to branch branch-0.6
in repository https://gitbox.apache.org/repos/asf/incubator-uniffle.git


The following commit(s) were added to refs/heads/branch-0.6 by this push:
 new 17dd9d78 Fix incorrect log format strings (#220)
17dd9d78 is described below

commit 17dd9d787c0c371630aa7cf1d49ef13cd3b14a9a
Author: Kaijie Chen 
AuthorDate: Thu Sep 15 18:43:18 2022 +0800

Fix incorrect log format strings (#220)

### What changes were proposed in this pull request?

Fix incorrect log format strings.

### Why are the changes needed?

SLF4J use `{}` as placeholder in format strings.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

CI.
---
 .../apache/uniffle/coordinator/AppBalanceSelectStorageStrategy.java   | 4 ++--
 .../uniffle/coordinator/LowestIOSampleCostSelectStorageStrategy.java  | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/coordinator/src/main/java/org/apache/uniffle/coordinator/AppBalanceSelectStorageStrategy.java
 
b/coordinator/src/main/java/org/apache/uniffle/coordinator/AppBalanceSelectStorageStrategy.java
index c42da449..b77c7f93 100644
--- 
a/coordinator/src/main/java/org/apache/uniffle/coordinator/AppBalanceSelectStorageStrategy.java
+++ 
b/coordinator/src/main/java/org/apache/uniffle/coordinator/AppBalanceSelectStorageStrategy.java
@@ -90,7 +90,7 @@ public class AppBalanceSelectStorageStrategy implements 
SelectStorageStrategy {
 } else {
   // it may be happened when assignment remote storage
   // and refresh remote storage at the same time
-  LOG.warn("Remote storage path lost during assignment: %s doesn't exist, 
reset it to 1",
+  LOG.warn("Remote storage path lost during assignment: {} doesn't exist, 
reset it to 1",
   remoteStoragePath);
   remoteStoragePathCounter.put(remoteStoragePath, new RankValue(1));
 }
@@ -104,7 +104,7 @@ public class AppBalanceSelectStorageStrategy implements 
SelectStorageStrategy {
   if (atomic != null) {
 double count = atomic.getAppNum().decrementAndGet();
 if (count < 0) {
-  LOG.warn("Unexpected counter for remote storage: %s, which is %i, 
reset to 0",
+  LOG.warn("Unexpected counter for remote storage: {}, which is {}, 
reset to 0",
   storagePath, count);
   atomic.getAppNum().set(0);
 }
diff --git 
a/coordinator/src/main/java/org/apache/uniffle/coordinator/LowestIOSampleCostSelectStorageStrategy.java
 
b/coordinator/src/main/java/org/apache/uniffle/coordinator/LowestIOSampleCostSelectStorageStrategy.java
index 96147b8e..f55e04ce 100644
--- 
a/coordinator/src/main/java/org/apache/uniffle/coordinator/LowestIOSampleCostSelectStorageStrategy.java
+++ 
b/coordinator/src/main/java/org/apache/uniffle/coordinator/LowestIOSampleCostSelectStorageStrategy.java
@@ -177,7 +177,7 @@ public class LowestIOSampleCostSelectStorageStrategy 
implements SelectStorageStr
   remoteStoragePathRankValue.put(remoteStoragePath, new RankValue(1));
   // it may be happened when assignment remote storage
   // and refresh remote storage at the same time
-  LOG.warn("Remote storage path lost during assignment: %s doesn't exist, "
+  LOG.warn("Remote storage path lost during assignment: {} doesn't exist, "
   + "reset the rank value to 0 and app size to 1.", remoteStoragePath);
 }
   }
@@ -190,7 +190,7 @@ public class LowestIOSampleCostSelectStorageStrategy 
implements SelectStorageStr
   if (atomic != null) {
 double count = atomic.getAppNum().decrementAndGet();
 if (count < 0) {
-  LOG.warn("Unexpected counter for remote storage: %s, which is %i, 
reset to 0",
+  LOG.warn("Unexpected counter for remote storage: {}, which is {}, 
reset to 0",
   storagePath, count);
   atomic.getAppNum().set(0);
 }



[incubator-uniffle] branch master updated: Fix incorrect log format strings (#220)

2022-09-15 Thread roryqi
This is an automated email from the ASF dual-hosted git repository.

roryqi pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-uniffle.git


The following commit(s) were added to refs/heads/master by this push:
 new 20aa41d0 Fix incorrect log format strings (#220)
20aa41d0 is described below

commit 20aa41d0a4eb843269feab88cd453330a28ba0d3
Author: Kaijie Chen 
AuthorDate: Thu Sep 15 18:43:18 2022 +0800

Fix incorrect log format strings (#220)

### What changes were proposed in this pull request?

Fix incorrect log format strings.

### Why are the changes needed?

SLF4J use `{}` as placeholder in format strings.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

CI.
---
 .../apache/uniffle/coordinator/AppBalanceSelectStorageStrategy.java   | 4 ++--
 .../uniffle/coordinator/LowestIOSampleCostSelectStorageStrategy.java  | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/coordinator/src/main/java/org/apache/uniffle/coordinator/AppBalanceSelectStorageStrategy.java
 
b/coordinator/src/main/java/org/apache/uniffle/coordinator/AppBalanceSelectStorageStrategy.java
index c42da449..b77c7f93 100644
--- 
a/coordinator/src/main/java/org/apache/uniffle/coordinator/AppBalanceSelectStorageStrategy.java
+++ 
b/coordinator/src/main/java/org/apache/uniffle/coordinator/AppBalanceSelectStorageStrategy.java
@@ -90,7 +90,7 @@ public class AppBalanceSelectStorageStrategy implements 
SelectStorageStrategy {
 } else {
   // it may be happened when assignment remote storage
   // and refresh remote storage at the same time
-  LOG.warn("Remote storage path lost during assignment: %s doesn't exist, 
reset it to 1",
+  LOG.warn("Remote storage path lost during assignment: {} doesn't exist, 
reset it to 1",
   remoteStoragePath);
   remoteStoragePathCounter.put(remoteStoragePath, new RankValue(1));
 }
@@ -104,7 +104,7 @@ public class AppBalanceSelectStorageStrategy implements 
SelectStorageStrategy {
   if (atomic != null) {
 double count = atomic.getAppNum().decrementAndGet();
 if (count < 0) {
-  LOG.warn("Unexpected counter for remote storage: %s, which is %i, 
reset to 0",
+  LOG.warn("Unexpected counter for remote storage: {}, which is {}, 
reset to 0",
   storagePath, count);
   atomic.getAppNum().set(0);
 }
diff --git 
a/coordinator/src/main/java/org/apache/uniffle/coordinator/LowestIOSampleCostSelectStorageStrategy.java
 
b/coordinator/src/main/java/org/apache/uniffle/coordinator/LowestIOSampleCostSelectStorageStrategy.java
index 96147b8e..f55e04ce 100644
--- 
a/coordinator/src/main/java/org/apache/uniffle/coordinator/LowestIOSampleCostSelectStorageStrategy.java
+++ 
b/coordinator/src/main/java/org/apache/uniffle/coordinator/LowestIOSampleCostSelectStorageStrategy.java
@@ -177,7 +177,7 @@ public class LowestIOSampleCostSelectStorageStrategy 
implements SelectStorageStr
   remoteStoragePathRankValue.put(remoteStoragePath, new RankValue(1));
   // it may be happened when assignment remote storage
   // and refresh remote storage at the same time
-  LOG.warn("Remote storage path lost during assignment: %s doesn't exist, "
+  LOG.warn("Remote storage path lost during assignment: {} doesn't exist, "
   + "reset the rank value to 0 and app size to 1.", remoteStoragePath);
 }
   }
@@ -190,7 +190,7 @@ public class LowestIOSampleCostSelectStorageStrategy 
implements SelectStorageStr
   if (atomic != null) {
 double count = atomic.getAppNum().decrementAndGet();
 if (count < 0) {
-  LOG.warn("Unexpected counter for remote storage: %s, which is %i, 
reset to 0",
+  LOG.warn("Unexpected counter for remote storage: {}, which is {}, 
reset to 0",
   storagePath, count);
   atomic.getAppNum().set(0);
 }



[incubator-uniffle] branch master updated: Add more badges in README (#219)

2022-09-15 Thread jshao
This is an automated email from the ASF dual-hosted git repository.

jshao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-uniffle.git


The following commit(s) were added to refs/heads/master by this push:
 new 3b210c0c Add more badges in README (#219)
3b210c0c is described below

commit 3b210c0cf2ea5e9cd23ce759a267c6c5b3eb302d
Author: Kaijie Chen 
AuthorDate: Thu Sep 15 15:51:43 2022 +0800

Add more badges in README (#219)
---
 README.md | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/README.md b/README.md
index e3521acc..8794e016 100644
--- a/README.md
+++ b/README.md
@@ -24,6 +24,9 @@ Currently it supports [Apache 
Spark](https://spark.apache.org) and [MapReduce](h
 
 
[![Build](https://github.com/apache/incubator-uniffle/actions/workflows/build.yml/badge.svg?branch=master=push)](https://github.com/apache/incubator-uniffle/actions/workflows/build.yml)
 
[![Codecov](https://codecov.io/gh/apache/incubator-uniffle/branch/master/graph/badge.svg)](https://codecov.io/gh/apache/incubator-uniffle)
+[![Total 
Lines](https://img.shields.io/tokei/lines/github/apache/incubator-uniffle)](https://github.com/apache/incubator-uniffle)
+[![Code 
Quality](https://img.shields.io/lgtm/grade/java/github/apache/incubator-uniffle?label=code%20quality)](https://lgtm.com/projects/g/apache/incubator-uniffle/)
+[![License](https://img.shields.io/github/license/apache/incubator-uniffle)](https://github.com/apache/incubator-uniffle/blob/master/LICENSE)
 
 ## Architecture
 ![Rss Architecture](docs/asset/rss_architecture.png)



[incubator-uniffle] branch branch-0.6 created (now dcdf8ae5)

2022-09-15 Thread roryqi
This is an automated email from the ASF dual-hosted git repository.

roryqi pushed a change to branch branch-0.6
in repository https://gitbox.apache.org/repos/asf/incubator-uniffle.git


  at dcdf8ae5 Add Notice and DISCLAMER file (#215)

No new revisions were added by this update.