You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2021/12/14 02:41:09 UTC

[kudu] branch master updated (44e5175 -> ea67260)

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

alexey pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git.


    from 44e5175  [java] bump log4j up to 2.15.0 version
     new 60d34b6  [KUDU-1959] - Fix the counter in StartupProgressStepsRemainingMetric()
     new ea67260  [java] bump log4j up to 2.16.0 version

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:
 java/gradle/dependencies.gradle         | 2 +-
 src/kudu/server/startup_path_handler.cc | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)

[kudu] 02/02: [java] bump log4j up to 2.16.0 version

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

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git

commit ea67260aad998db7d34a94d25261e121a668faec
Author: Alexey Serbin <al...@apache.org>
AuthorDate: Mon Dec 13 17:22:16 2021 -0800

    [java] bump log4j up to 2.16.0 version
    
    OK, it seems 2.16.0 is the new shiny version to have once the recent
    security vulnerability CVE-2021-44228 has been fixed in 2.15.0.
    
    Release notes for the new version of the package is available at [1].
    
    This is a follow-up to a6079a063c8f38166d91956ad46a4ce695a08019.
    
    [1] https://logging.apache.org/log4j/2.x/changes-report.html#a2.16.0
    
    Change-Id: Ied261b7fdb412839b40a0d68a131963c61b09356
    Reviewed-on: http://gerrit.cloudera.org:8080/18095
    Reviewed-by: Andrew Wong <aw...@cloudera.com>
    Tested-by: Kudu Jenkins
---
 java/gradle/dependencies.gradle | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/java/gradle/dependencies.gradle b/java/gradle/dependencies.gradle
index e3a9fa1..d03d93b 100755
--- a/java/gradle/dependencies.gradle
+++ b/java/gradle/dependencies.gradle
@@ -45,7 +45,7 @@ versions += [
     jmh            : "1.28",
     jsr305         : "3.0.2",
     junit          : "4.13.2",
-    log4j          : "2.15.0",
+    log4j          : "2.16.0",
     micrometer     : "1.6.5",
     mockito        : "3.8.0",
     murmur         : "1.0.0",

[kudu] 01/02: [KUDU-1959] - Fix the counter in StartupProgressStepsRemainingMetric()

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

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git

commit 60d34b68f4c42da04dd0a064db135568c5d75af9
Author: Abhishek Chennaka <ac...@cloudera.com>
AuthorDate: Mon Dec 6 17:43:13 2021 -0500

    [KUDU-1959] - Fix the counter in StartupProgressStepsRemainingMetric()
    
    The counter was incremented twice if tablets are not processed
    during the startup of a tablet server.
    
    This is a follow-up to 59070bf.
    
    Change-Id: I6570f438dd85aafa16093465ae654ece8d056eb5
    Reviewed-on: http://gerrit.cloudera.org:8080/18073
    Reviewed-by: Alexey Serbin <as...@cloudera.com>
    Tested-by: Kudu Jenkins
---
 src/kudu/server/startup_path_handler.cc | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/kudu/server/startup_path_handler.cc b/src/kudu/server/startup_path_handler.cc
index 6e1948f..e059c1f 100644
--- a/src/kudu/server/startup_path_handler.cc
+++ b/src/kudu/server/startup_path_handler.cc
@@ -141,7 +141,6 @@ int StartupPathHandler::StartupProgressStepsRemainingMetric() {
   int counter = 0;
   counter += (init_progress_.IsStopped() ? 0 : 1);
   counter += (read_filesystem_progress_.IsStopped() ? 0 : 1);
-  counter += (is_tablet_server_ ? (start_tablets_progress_.IsStopped() ? 0 : 1) : 0);
   if (is_tablet_server_) {
     counter += start_tablets_progress_.IsStopped() ? 0 : 1;
   } else {