[incubator-uniffle] branch master updated: Change url of total lines badge in README (#222)
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)
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)
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)
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)
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.