[hadoop-ozone] branch master updated (571d4a3 -> a731eea)

2019-11-21 Thread dineshc
This is an automated email from the ASF dual-hosted git repository.

dineshc pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git.


from 571d4a3  HDDS-2605. Use LongSupplier to avoid boxing (#256)
 add a731eea  HDDS-2603. Avoid unnecessary boxing in XceiverClientReply 
(#255)

No new revisions were added by this update.

Summary of changes:
 .../org/apache/hadoop/hdds/scm/XceiverClientReply.java | 18 +-
 1 file changed, 5 insertions(+), 13 deletions(-)


-
To unsubscribe, e-mail: ozone-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-commits-h...@hadoop.apache.org



[hadoop-ozone] branch master updated (7d35218 -> 571d4a3)

2019-11-21 Thread dineshc
This is an automated email from the ASF dual-hosted git repository.

dineshc pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git.


from 7d35218  HDDS-2394. Ozone S3 Gateway allows bucket name with 
underscore to be created (#243)
 add 571d4a3  HDDS-2605. Use LongSupplier to avoid boxing (#256)

No new revisions were added by this update.

Summary of changes:
 .../org/apache/hadoop/ozone/freon/ProgressBar.java | 33 --
 .../hadoop/ozone/freon/RandomKeyGenerator.java |  4 +--
 .../apache/hadoop/ozone/freon/TestProgressBar.java | 24 ++--
 3 files changed, 28 insertions(+), 33 deletions(-)


-
To unsubscribe, e-mail: ozone-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-commits-h...@hadoop.apache.org



[hadoop-ozone] branch HDDS-2587 updated: Add build step (#252)

2019-11-21 Thread bharat
This is an automated email from the ASF dual-hosted git repository.

bharat pushed a commit to branch HDDS-2587
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git


The following commit(s) were added to refs/heads/HDDS-2587 by this push:
 new ecdd884  Add build step (#252)
ecdd884 is described below

commit ecdd88420ce67edbadcdedc2f30f294df0093504
Author: Vivek Ratnavel Subramanian 
AuthorDate: Thu Nov 21 11:26:57 2019 -0800

Add build step (#252)
---
 .github/workflows/post-commit.yml | 8 
 .github/workflows/pr.yml  | 8 
 2 files changed, 16 insertions(+)

diff --git a/.github/workflows/post-commit.yml 
b/.github/workflows/post-commit.yml
index ef6237f..b10d270 100644
--- a/.github/workflows/post-commit.yml
+++ b/.github/workflows/post-commit.yml
@@ -16,6 +16,14 @@ name: build-branch
 on:
   - push
 jobs:
+  build:
+name: compile
+runs-on: ubuntu-18.04
+steps:
+  - uses: actions/checkout@master
+  - uses: ./.github/buildenv
+with:
+  args: ./hadoop-ozone/dev-support/checks/build.sh
   rat:
 name: rat
 runs-on: ubuntu-18.04
diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml
index 2662ee4..eb39fad 100644
--- a/.github/workflows/pr.yml
+++ b/.github/workflows/pr.yml
@@ -16,6 +16,14 @@ name: pr-check
 on:
   - pull_request
 jobs:
+  build:
+name: compile
+runs-on: ubuntu-18.04
+steps:
+  - uses: actions/checkout@master
+  - uses: ./.github/buildenv
+with:
+  args: ./hadoop-ozone/dev-support/checks/build.sh
   rat:
 name: rat
 runs-on: ubuntu-18.04


-
To unsubscribe, e-mail: ozone-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-commits-h...@hadoop.apache.org



[hadoop-ozone] branch master updated (d29838b -> 7d35218)

2019-11-21 Thread bharat
This is an automated email from the ASF dual-hosted git repository.

bharat pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git.


from d29838b  HDDS-2536. Add ozone.om.internal.service.id to OM HA 
configuration. (#218)
 add 7d35218  HDDS-2394. Ozone S3 Gateway allows bucket name with 
underscore to be created (#243)

No new revisions were added by this update.

Summary of changes:
 .../apache/hadoop/ozone/client/rpc/RpcClient.java|  7 +++
 .../hadoop/ozone/om/exceptions/OMException.java  |  4 +++-
 .../common/src/main/proto/OzoneManagerProtocol.proto |  1 +
 .../dist/src/main/smoketest/s3/bucketcreate.robot|  4 
 .../hadoop/ozone/s3/endpoint/BucketEndpoint.java | 20 ++--
 .../hadoop/ozone/s3/exception/S3ErrorTable.java  |  4 
 6 files changed, 33 insertions(+), 7 deletions(-)


-
To unsubscribe, e-mail: ozone-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-commits-h...@hadoop.apache.org



[hadoop-ozone] branch HDDS-2587 updated (b42d9c2 -> 36bc4df)

2019-11-21 Thread elek
This is an automated email from the ASF dual-hosted git repository.

elek pushed a change to branch HDDS-2587
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git.


from b42d9c2  separate PR and main branch build definition
 new 9d3a845  coverage report is turned on
 new 0d9a830  Sonar is executed with maven batch mode
 new 36bc4df  fix unit test

The 3 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:
 hadoop-hdds/pom.xml  |  3 ++-
 hadoop-ozone/dev-support/checks/sonar.sh |  2 +-
 hadoop-ozone/dev-support/checks/unit.sh  |  2 +-
 hadoop-ozone/pom.xml |  3 ++-
 pom.xml  | 20 +++-
 5 files changed, 25 insertions(+), 5 deletions(-)


-
To unsubscribe, e-mail: ozone-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-commits-h...@hadoop.apache.org



[hadoop-ozone] 03/03: fix unit test

2019-11-21 Thread elek
This is an automated email from the ASF dual-hosted git repository.

elek pushed a commit to branch HDDS-2587
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git

commit 36bc4df6a9719c650a68a25b17feeec9ccde19f7
Author: Márton Elek 
AuthorDate: Thu Nov 21 17:15:16 2019 +0100

fix unit test
---
 hadoop-ozone/dev-support/checks/unit.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hadoop-ozone/dev-support/checks/unit.sh 
b/hadoop-ozone/dev-support/checks/unit.sh
index 585375e..adfef53 100755
--- a/hadoop-ozone/dev-support/checks/unit.sh
+++ b/hadoop-ozone/dev-support/checks/unit.sh
@@ -17,7 +17,7 @@ DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null 2>&1 
&& pwd )"
 cd "$DIR/../../.." || exit 1
 
 export MAVEN_OPTS="-Xmx4096m"
-mvn -B -DskipShade -Dskip.yarn -fn test -pl 
\!:hadoop-ozone-integration-test,\!:hadoop-ozone-filesystem,\!:hadoop-ozone-tools
 "$@"
+mvn -B -DskipShade -Dskip.yarn -fn test -pl 
\!:hadoop-ozone-integration-test,\!:hadoop-ozone-tools "$@"
 
 REPORT_DIR=${OUTPUT_DIR:-"$DIR/../../../target/unit"}
 mkdir -p "$REPORT_DIR"


-
To unsubscribe, e-mail: ozone-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-commits-h...@hadoop.apache.org



[hadoop-ozone] 01/03: coverage report is turned on

2019-11-21 Thread elek
This is an automated email from the ASF dual-hosted git repository.

elek pushed a commit to branch HDDS-2587
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git

commit 9d3a84518a3e74dcd9598a7672e13ee20062313b
Author: Márton Elek 
AuthorDate: Thu Nov 21 15:43:15 2019 +0100

coverage report is turned on
---
 hadoop-hdds/pom.xml |  3 ++-
 hadoop-ozone/dev-support/checks/unit.sh |  2 +-
 hadoop-ozone/pom.xml|  3 ++-
 pom.xml | 20 +++-
 4 files changed, 24 insertions(+), 4 deletions(-)

diff --git a/hadoop-hdds/pom.xml b/hadoop-hdds/pom.xml
index 9cd5db3..882755a 100644
--- a/hadoop-hdds/pom.xml
+++ b/hadoop-hdds/pom.xml
@@ -369,7 +369,8 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd";>
 
   ${testsThreadCount}
   false
-  ${maven-surefire-plugin.argLine} 
-DminiClusterDedicatedDirs=true
+  
+  ${maven-surefire-plugin.argLine}  @{argLine} 
-DminiClusterDedicatedDirs=true
   
 ${testsThreadCount}
 
${test.build.data}/${surefire.forkNumber}
diff --git a/hadoop-ozone/dev-support/checks/unit.sh 
b/hadoop-ozone/dev-support/checks/unit.sh
index 44b6d0b..585375e 100755
--- a/hadoop-ozone/dev-support/checks/unit.sh
+++ b/hadoop-ozone/dev-support/checks/unit.sh
@@ -17,7 +17,7 @@ DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null 2>&1 
&& pwd )"
 cd "$DIR/../../.." || exit 1
 
 export MAVEN_OPTS="-Xmx4096m"
-mvn -B -fn test -pl 
\!:hadoop-ozone-integration-test,\!:hadoop-ozone-filesystem,\!:hadoop-ozone-tools
 "$@"
+mvn -B -DskipShade -Dskip.yarn -fn test -pl 
\!:hadoop-ozone-integration-test,\!:hadoop-ozone-filesystem,\!:hadoop-ozone-tools
 "$@"
 
 REPORT_DIR=${OUTPUT_DIR:-"$DIR/../../../target/unit"}
 mkdir -p "$REPORT_DIR"
diff --git a/hadoop-ozone/pom.xml b/hadoop-ozone/pom.xml
index 6d8b465..8a19298 100644
--- a/hadoop-ozone/pom.xml
+++ b/hadoop-ozone/pom.xml
@@ -397,7 +397,8 @@
 
   ${testsThreadCount}
   false
-  ${maven-surefire-plugin.argLine} 
-DminiClusterDedicatedDirs=true
+  
+  ${maven-surefire-plugin.argLine} @{argLine} 
-DminiClusterDedicatedDirs=true
   
 ${testsThreadCount}
 
${test.build.data}/${surefire.forkNumber}
diff --git a/pom.xml b/pom.xml
index d6e2da6..c8b16c1 100644
--- a/pom.xml
+++ b/pom.xml
@@ -1670,6 +1670,11 @@ xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 
http://maven.apache.org/xs
   apache-rat-plugin
   ${apache-rat-plugin.version}
 
+
+  org.jacoco
+  jacoco-maven-plugin
+  0.8.3
+
   
 
 
@@ -1760,12 +1765,25 @@ xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 
http://maven.apache.org/xs
 
   
   
+org.jacoco
+jacoco-maven-plugin
+
+  
+default-prepare-agent
+
+  prepare-agent
+
+  
+
+  
+  
 org.apache.maven.plugins
 maven-surefire-plugin
 
   false
   
${surefire.fork.timeout}
-  ${maven-surefire-plugin.argLine}
+  
+  ${maven-surefire-plugin.argLine} @{argLine}
   
 ${hadoop.common.build.dir}
 


-
To unsubscribe, e-mail: ozone-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-commits-h...@hadoop.apache.org



[hadoop-ozone] 02/03: Sonar is executed with maven batch mode

2019-11-21 Thread elek
This is an automated email from the ASF dual-hosted git repository.

elek pushed a commit to branch HDDS-2587
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git

commit 0d9a83087a00cf89c350178bb5a7c3154f4f1130
Author: Márton Elek 
AuthorDate: Thu Nov 21 15:43:51 2019 +0100

Sonar is executed with maven batch mode
---
 hadoop-ozone/dev-support/checks/sonar.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hadoop-ozone/dev-support/checks/sonar.sh 
b/hadoop-ozone/dev-support/checks/sonar.sh
index 0282b15..55f219b 100755
--- a/hadoop-ozone/dev-support/checks/sonar.sh
+++ b/hadoop-ozone/dev-support/checks/sonar.sh
@@ -20,4 +20,4 @@ if [ ! "$SONAR_TOKEN" ]; then
   echo "SONAR_TOKEN environment variable should be set"
   exit 1
 fi
-mvn verify -DskipShade -DskipTests -Dskip.yarn 
org.sonarsource.scanner.maven:sonar-maven-plugin:3.6.0.1398:sonar 
-Dsonar.host.url=https://sonarcloud.io -Dsonar.organization=apache 
-Dsonar.projectKey=hadoop-ozone
+mvn -B verify -DskipShade -DskipTests -Dskip.yarn 
org.sonarsource.scanner.maven:sonar-maven-plugin:3.6.0.1398:sonar 
-Dsonar.host.url=https://sonarcloud.io -Dsonar.organization=apache 
-Dsonar.projectKey=hadoop-ozone


-
To unsubscribe, e-mail: ozone-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-commits-h...@hadoop.apache.org



[hadoop-ozone] 01/02: revert files added by a merge to this branch

2019-11-21 Thread elek
This is an automated email from the ASF dual-hosted git repository.

elek pushed a commit to branch HDDS-2587
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git

commit d1dfbc4cb3102b6687e97a527e9d67a13d0bda8c
Author: Márton Elek 
AuthorDate: Thu Nov 21 15:23:37 2019 +0100

revert files added by a merge to this branch
---
 .../hdds/scm/client/ContainerOperationClient.java  |   2 +-
 .../hadoop/hdds/scm/client/HddsClientUtils.java|   5 +
 .../apache/hadoop/hdds/scm/storage/BufferPool.java |   2 +-
 .../hadoop/hdds/scm/storage/CommitWatcher.java |   4 +-
 .../hadoop/hdds/scm/net/NodeSchemaLoader.java  |  27 +-
 .../apache/hadoop/hdds/tracing/TraceAllMethod.java |   5 -
 .../common/src/main/resources/ozone-default.xml|  11 +-
 .../hadoop/hdds/scm/net/TestNodeSchemaLoader.java  | 151 
 .../common/src/test/resources/log4j.properties |  23 --
 .../networkTopologyTestFiles/external-entity.xml   |  52 ---
 .../hadoop/hdds/conf/ConfigFileAppender.java   |  21 +-
 .../hdds/scm/server/StorageContainerManager.java   |  74 ++--
 .../org/apache/hadoop/ozone/om/OMConfigKeys.java   |   3 -
 .../hadoop/ozone/om/exceptions/OMException.java|   6 +-
 .../src/main/proto/OzoneManagerProtocol.proto  |   1 -
 .../hadoop/ozone/MiniOzoneHAClusterImpl.java   |   6 +-
 .../hadoop/ozone/TestSecureOzoneCluster.java   | 399 -
 .../hadoop/ozone/client/rpc/TestCommitWatcher.java |   9 +-
 .../apache/hadoop/ozone/om/TestKeyManagerImpl.java | 133 +--
 .../org/apache/hadoop/ozone/om/KeyManagerImpl.java |  66 ++--
 .../apache/hadoop/ozone/om/ha/OMHANodeDetails.java |  21 +-
 .../ozone/web/ozShell/keys/GetKeyHandler.java  |  44 +--
 .../ozone/web/ozShell/keys/PutKeyHandler.java  |  35 +-
 .../endpoint/MultiDeleteRequestUnmarshaller.java   |   8 +-
 .../ozone/s3/util/RangeHeaderParserUtil.java   |   4 +-
 .../ozone/s3/util/TestRangeHeaderParserUtil.java   |   6 +-
 .../ozone/audit/parser/common/DatabaseHelper.java  | 157 
 .../audit/parser/handler/LoadCommandHandler.java   |  11 +-
 .../audit/parser/handler/QueryCommandHandler.java  |  11 +-
 .../parser/handler/TemplateCommandHandler.java |  11 +-
 30 files changed, 554 insertions(+), 754 deletions(-)

diff --git 
a/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/ContainerOperationClient.java
 
b/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/ContainerOperationClient.java
index 6e2fd59..b1d4538 100644
--- 
a/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/ContainerOperationClient.java
+++ 
b/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/ContainerOperationClient.java
@@ -221,7 +221,7 @@ public class ContainerOperationClient implements ScmClient {
 // That makes sense, but it is not needed for the client to work.
 if (LOG.isDebugEnabled()) {
   LOG.debug("Pipeline creation successful. Pipeline: {}",
-  pipeline);
+  pipeline.toString());
 }
   }
 
diff --git 
a/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/HddsClientUtils.java
 
b/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/HddsClientUtils.java
index d7e3f4d..8c9a11a 100644
--- 
a/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/HddsClientUtils.java
+++ 
b/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/HddsClientUtils.java
@@ -49,6 +49,8 @@ import org.apache.ratis.protocol.AlreadyClosedException;
 import org.apache.ratis.protocol.GroupMismatchException;
 import org.apache.ratis.protocol.NotReplicatedException;
 import org.apache.ratis.protocol.RaftRetryFailureException;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import java.io.IOException;
 import java.net.InetSocketAddress;
@@ -74,6 +76,9 @@ import java.util.concurrent.TimeoutException;
 @InterfaceStability.Unstable
 public final class HddsClientUtils {
 
+  private static final Logger LOG = LoggerFactory.getLogger(
+  HddsClientUtils.class);
+
   private HddsClientUtils() {
   }
 
diff --git 
a/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/storage/BufferPool.java
 
b/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/storage/BufferPool.java
index f57700e..f986fe3 100644
--- 
a/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/storage/BufferPool.java
+++ 
b/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/storage/BufferPool.java
@@ -94,7 +94,7 @@ public class BufferPool {
 // always remove from head of the list and append at last
 final ChunkBuffer buffer = bufferList.remove(0);
 // Ensure the buffer to be removed is always at the head of the list.
-Preconditions.checkArgument(buffer == chunkBuffer);
+Preconditions.checkArgument(buffer.equals(chunkBuffer));
 buffer.clear();
 bufferList.add(buffer);
 Preconditions.checkArgument(currentBufferIndex >= 0);
diff --git 
a/hadoop-hdds/client/src/main/j

[hadoop-ozone] branch HDDS-2587 updated (2f27d93 -> b42d9c2)

2019-11-21 Thread elek
This is an automated email from the ASF dual-hosted git repository.

elek pushed a change to branch HDDS-2587
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git.


from 2f27d93  retrigger build
 new d1dfbc4  revert files added by a merge to this branch
 new b42d9c2  separate PR and main branch build definition

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:
 .github/workflows/main.yml | 121 ---
 .github/workflows/post-commit.yml  | 112 ++
 .github/workflows/pr.yml   | 106 ++
 .../hdds/scm/client/ContainerOperationClient.java  |   2 +-
 .../hadoop/hdds/scm/client/HddsClientUtils.java|   5 +
 .../apache/hadoop/hdds/scm/storage/BufferPool.java |   2 +-
 .../hadoop/hdds/scm/storage/CommitWatcher.java |   4 +-
 .../hadoop/hdds/scm/net/NodeSchemaLoader.java  |  27 +-
 .../apache/hadoop/hdds/tracing/TraceAllMethod.java |   5 -
 .../common/src/main/resources/ozone-default.xml|  11 +-
 .../hadoop/hdds/scm/net/TestNodeSchemaLoader.java  | 151 
 .../common/src/test/resources/log4j.properties |  23 --
 .../networkTopologyTestFiles/external-entity.xml   |  52 ---
 .../hadoop/hdds/conf/ConfigFileAppender.java   |  21 +-
 .../hdds/scm/server/StorageContainerManager.java   |  74 ++--
 .../org/apache/hadoop/ozone/om/OMConfigKeys.java   |   3 -
 .../hadoop/ozone/om/exceptions/OMException.java|   6 +-
 .../src/main/proto/OzoneManagerProtocol.proto  |   1 -
 hadoop-ozone/dev-support/checks/sonar.sh   |   2 +-
 .../hadoop/ozone/MiniOzoneHAClusterImpl.java   |   6 +-
 .../hadoop/ozone/TestSecureOzoneCluster.java   | 399 -
 .../hadoop/ozone/client/rpc/TestCommitWatcher.java |   9 +-
 .../apache/hadoop/ozone/om/TestKeyManagerImpl.java | 133 +--
 .../org/apache/hadoop/ozone/om/KeyManagerImpl.java |  66 ++--
 .../apache/hadoop/ozone/om/ha/OMHANodeDetails.java |  21 +-
 .../ozone/web/ozShell/keys/GetKeyHandler.java  |  44 +--
 .../ozone/web/ozShell/keys/PutKeyHandler.java  |  35 +-
 .../endpoint/MultiDeleteRequestUnmarshaller.java   |   8 +-
 .../ozone/s3/util/RangeHeaderParserUtil.java   |   4 +-
 .../ozone/s3/util/TestRangeHeaderParserUtil.java   |   6 +-
 .../ozone/audit/parser/common/DatabaseHelper.java  | 157 
 .../audit/parser/handler/LoadCommandHandler.java   |  11 +-
 .../audit/parser/handler/QueryCommandHandler.java  |  11 +-
 .../parser/handler/TemplateCommandHandler.java |  11 +-
 34 files changed, 773 insertions(+), 876 deletions(-)
 delete mode 100644 .github/workflows/main.yml
 create mode 100644 .github/workflows/post-commit.yml
 create mode 100644 .github/workflows/pr.yml
 delete mode 100644 hadoop-hdds/common/src/test/resources/log4j.properties
 delete mode 100644 
hadoop-hdds/common/src/test/resources/networkTopologyTestFiles/external-entity.xml


-
To unsubscribe, e-mail: ozone-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-commits-h...@hadoop.apache.org



[hadoop-ozone] 02/02: separate PR and main branch build definition

2019-11-21 Thread elek
This is an automated email from the ASF dual-hosted git repository.

elek pushed a commit to branch HDDS-2587
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git

commit b42d9c26080649596857941ed5372e1e5f4294aa
Author: Márton Elek 
AuthorDate: Thu Nov 21 15:35:00 2019 +0100

separate PR and main branch build definition
---
 .github/workflows/main.yml   | 121 ---
 .github/workflows/post-commit.yml| 112 
 .github/workflows/pr.yml | 106 +++
 hadoop-ozone/dev-support/checks/sonar.sh |   2 +-
 4 files changed, 219 insertions(+), 122 deletions(-)

diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
deleted file mode 100644
index 2b52480..000
--- a/.github/workflows/main.yml
+++ /dev/null
@@ -1,121 +0,0 @@
-# Licensed to the Apache Software Foundation (ASF) under one or more
-# contributor license agreements.  See the NOTICE file distributed with
-# this work for additional information regarding copyright ownership.
-# The ASF licenses this file to You under the Apache License, Version 2.0
-# (the "License"); you may not use this file except in compliance with
-# the License.  You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-name: Build
-on:
-  - push
-  - pull_request
-jobs:
-#  build:
-#name: compile
-#runs-on: ubuntu-18.04
-#steps:
-#- uses: actions/checkout@master
-#- uses: ./.github/buildenv
-#  with:
-# args: ./hadoop-ozone/dev-support/checks/build.sh
-#  rat:
-#name: rat
-#runs-on: ubuntu-18.04
-#steps:
-#- uses: actions/checkout@master
-#- uses: ./.github/buildenv
-#  with:
-# args: ./hadoop-ozone/dev-support/checks/rat.sh
-#- uses: actions/upload-artifact@master
-#  if: always()
-#  with:
-#name: rat
-#path: target/rat
-#  author:
-#name: author
-#runs-on: ubuntu-18.04
-#steps:
-#- uses: actions/checkout@master
-#- uses: ./.github/buildenv
-#  with:
-# args: ./hadoop-ozone/dev-support/checks/author.sh
-#- uses: actions/upload-artifact@master
-#  if: always()
-#  with:
-#name: author
-#path: target/author
-  sonar:
-name: sonar
-runs-on: ubuntu-18.04
-steps:
-- uses: actions/checkout@master
-#- uses: ./.github/buildenv
-#  with:
-# args: ./hadoop-ozone/dev-support/checks/build.sh
-- uses: ./.github/buildenv
-  with:
-args: ./hadoop-ozone/dev-support/checks/sonar.sh
-  env:
-SONAR_TOKEN: ${{ secrets.SONARCLOUD_TOKEN }}
-GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-- uses: actions/upload-artifact@master
-  if: always()
-  with:
-name: sonar
-path: target/sonar
-#  checkstyle:
-#name: checkstyle
-#runs-on: ubuntu-18.04
-#steps:
-#- uses: actions/checkout@master
-#- uses: ./.github/buildenv
-#  with:
-# args: ./hadoop-ozone/dev-support/checks/checkstyle.sh
-#- uses: actions/upload-artifact@master
-#  if: always()
-#  with:
-#name: checkstyle
-#path: target/checkstyle
-#  findbugs:
-#name: findbugs
-#runs-on: ubuntu-18.04
-#steps:
-#- uses: actions/checkout@master
-#- uses: ./.github/buildenv
-#  with:
-# args: ./hadoop-ozone/dev-support/checks/findbugs.sh
-#- uses: actions/upload-artifact@master
-#  if: always()
-#  with:
-#name: findbugs
-#path: target/findbugs
-#  acceptance:
-#name: acceptance
-#runs-on: ubuntu-18.04
-#needs:
-#   - build
-#   - rat
-#   - checkstyle
-#   - unit
-#   - findbugs
-#steps:
-#- uses: actions/checkout@master
-#- uses: ./.github/buildenv
-#  with:
-#args: ./hadoop-ozone/dev-support/checks/build.sh
-#- run: sudo pip install robotframework
-#- run: sudo chown runner -R .
-#- run: cd ./hadoop-ozone/dist/target/ozone-*-SNAPSHOT/ && mkdir .aws 
&& sudo chown 1000 .aws
-#- run: ./hadoop-ozone/dev-support/checks/acceptance.sh
-#- uses: actions/upload-artifact@master
-#  if: always()
-#  with:
-#name: acceptance
-#path: target/acceptance
diff --git a/.github/workflows/post-commit.yml 
b/.github/workflows/post-commit.yml
new file mode 100644
index 0

[hadoop-ozone] branch HDDS-2587 updated: retrigger build

2019-11-21 Thread elek
This is an automated email from the ASF dual-hosted git repository.

elek pushed a commit to branch HDDS-2587
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git


The following commit(s) were added to refs/heads/HDDS-2587 by this push:
 new 2f27d93  retrigger build
2f27d93 is described below

commit 2f27d93b87dc73c229115c90cb14008c5df11e3b
Author: Márton Elek 
AuthorDate: Thu Nov 21 11:03:10 2019 +0100

retrigger build


-
To unsubscribe, e-mail: ozone-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-commits-h...@hadoop.apache.org



[hadoop-ozone] branch HDDS-2587 updated: skip tests

2019-11-21 Thread elek
This is an automated email from the ASF dual-hosted git repository.

elek pushed a commit to branch HDDS-2587
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git


The following commit(s) were added to refs/heads/HDDS-2587 by this push:
 new c0e1710  skip tests
c0e1710 is described below

commit c0e17103d80690b46cac402ea8325ef202986f8b
Author: Márton Elek 
AuthorDate: Thu Nov 21 10:43:48 2019 +0100

skip tests
---
 hadoop-ozone/dev-support/checks/sonar.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hadoop-ozone/dev-support/checks/sonar.sh 
b/hadoop-ozone/dev-support/checks/sonar.sh
index 68edb7e..670a11d 100755
--- a/hadoop-ozone/dev-support/checks/sonar.sh
+++ b/hadoop-ozone/dev-support/checks/sonar.sh
@@ -20,4 +20,4 @@ if [ ! "$SONAR_TOKEN" ]; then
   echo "SONAR_TOKEN environment variable should be set"
   exit 1
 fi
-mvn install org.sonarsource.scanner.maven:sonar-maven-plugin:3.6.0.1398:sonar 
-Dsonar.host.url=https://sonarcloud.io -Dsonar.organization=apache 
-Dsonar.projectKey=hadoop-ozone
+mvn install -DskipTests 
org.sonarsource.scanner.maven:sonar-maven-plugin:3.6.0.1398:sonar 
-Dsonar.host.url=https://sonarcloud.io -Dsonar.organization=apache 
-Dsonar.projectKey=hadoop-ozone


-
To unsubscribe, e-mail: ozone-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-commits-h...@hadoop.apache.org



[hadoop-ozone] branch HDDS-2587 updated: install before sonar:sonar

2019-11-21 Thread elek
This is an automated email from the ASF dual-hosted git repository.

elek pushed a commit to branch HDDS-2587
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git


The following commit(s) were added to refs/heads/HDDS-2587 by this push:
 new 376856d  install before sonar:sonar
376856d is described below

commit 376856d9f430a3a4c5d13cc5ddfc325f1ceab139
Author: Márton Elek 
AuthorDate: Thu Nov 21 10:37:54 2019 +0100

install before sonar:sonar
---
 .github/workflows/main.yml   | 6 +++---
 hadoop-ozone/dev-support/checks/sonar.sh | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
index 0a1608a..2b52480 100644
--- a/.github/workflows/main.yml
+++ b/.github/workflows/main.yml
@@ -56,9 +56,9 @@ jobs:
 runs-on: ubuntu-18.04
 steps:
 - uses: actions/checkout@master
-- uses: ./.github/buildenv
-  with:
- args: ./hadoop-ozone/dev-support/checks/build.sh
+#- uses: ./.github/buildenv
+#  with:
+# args: ./hadoop-ozone/dev-support/checks/build.sh
 - uses: ./.github/buildenv
   with:
 args: ./hadoop-ozone/dev-support/checks/sonar.sh
diff --git a/hadoop-ozone/dev-support/checks/sonar.sh 
b/hadoop-ozone/dev-support/checks/sonar.sh
index 135a47e..68edb7e 100755
--- a/hadoop-ozone/dev-support/checks/sonar.sh
+++ b/hadoop-ozone/dev-support/checks/sonar.sh
@@ -20,4 +20,4 @@ if [ ! "$SONAR_TOKEN" ]; then
   echo "SONAR_TOKEN environment variable should be set"
   exit 1
 fi
-mvn org.sonarsource.scanner.maven:sonar-maven-plugin:3.6.0.1398:sonar 
-Dsonar.host.url=https://sonarcloud.io -Dsonar.organization=apache 
-Dsonar.projectKey=hadoop-ozone
+mvn install org.sonarsource.scanner.maven:sonar-maven-plugin:3.6.0.1398:sonar 
-Dsonar.host.url=https://sonarcloud.io -Dsonar.organization=apache 
-Dsonar.projectKey=hadoop-ozone


-
To unsubscribe, e-mail: ozone-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-commits-h...@hadoop.apache.org



[hadoop-ozone] branch HDDS-2587 updated: Add sonar java classes property in pom.xml (#246)

2019-11-21 Thread bharat
This is an automated email from the ASF dual-hosted git repository.

bharat pushed a commit to branch HDDS-2587
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git


The following commit(s) were added to refs/heads/HDDS-2587 by this push:
 new ed8cbd4  Add sonar java classes property in pom.xml (#246)
ed8cbd4 is described below

commit ed8cbd4236f20981a9c60fbc803b8c181998fdd5
Author: Vivek Ratnavel Subramanian 
AuthorDate: Thu Nov 21 00:54:09 2019 -0800

Add sonar java classes property in pom.xml (#246)
---
 .../hdds/scm/client/ContainerOperationClient.java  |   2 +-
 .../hadoop/hdds/scm/client/HddsClientUtils.java|   5 -
 .../apache/hadoop/hdds/scm/storage/BufferPool.java |   2 +-
 .../hadoop/hdds/scm/storage/CommitWatcher.java |   4 +-
 .../hadoop/hdds/scm/net/NodeSchemaLoader.java  |  27 +-
 .../apache/hadoop/hdds/tracing/TraceAllMethod.java |   5 +
 .../common/src/main/resources/ozone-default.xml|  11 +-
 .../hadoop/hdds/scm/net/TestNodeSchemaLoader.java  | 151 
 .../common/src/test/resources/log4j.properties |  23 ++
 .../networkTopologyTestFiles/external-entity.xml   |  52 +++
 .../hadoop/hdds/conf/ConfigFileAppender.java   |  21 +-
 .../hdds/scm/server/StorageContainerManager.java   |  74 ++--
 .../org/apache/hadoop/ozone/om/OMConfigKeys.java   |   3 +
 .../hadoop/ozone/om/exceptions/OMException.java|   6 +-
 .../src/main/proto/OzoneManagerProtocol.proto  |   1 +
 hadoop-ozone/dev-support/checks/sonar.sh   |   2 +-
 .../hadoop/ozone/MiniOzoneHAClusterImpl.java   |   6 +-
 .../hadoop/ozone/TestSecureOzoneCluster.java   | 399 +
 .../hadoop/ozone/client/rpc/TestCommitWatcher.java |   9 +-
 .../apache/hadoop/ozone/om/TestKeyManagerImpl.java | 133 ++-
 .../org/apache/hadoop/ozone/om/KeyManagerImpl.java |  66 ++--
 .../apache/hadoop/ozone/om/ha/OMHANodeDetails.java |  21 +-
 .../ozone/web/ozShell/keys/GetKeyHandler.java  |  44 ++-
 .../ozone/web/ozShell/keys/PutKeyHandler.java  |  35 +-
 .../endpoint/MultiDeleteRequestUnmarshaller.java   |   8 +-
 .../ozone/s3/util/RangeHeaderParserUtil.java   |   4 +-
 .../ozone/s3/util/TestRangeHeaderParserUtil.java   |   6 +-
 .../ozone/audit/parser/common/DatabaseHelper.java  | 157 
 .../audit/parser/handler/LoadCommandHandler.java   |  11 +-
 .../audit/parser/handler/QueryCommandHandler.java  |  11 +-
 .../parser/handler/TemplateCommandHandler.java |  11 +-
 pom.xml|   1 +
 32 files changed, 756 insertions(+), 555 deletions(-)

diff --git 
a/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/ContainerOperationClient.java
 
b/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/ContainerOperationClient.java
index b1d4538..6e2fd59 100644
--- 
a/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/ContainerOperationClient.java
+++ 
b/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/ContainerOperationClient.java
@@ -221,7 +221,7 @@ public class ContainerOperationClient implements ScmClient {
 // That makes sense, but it is not needed for the client to work.
 if (LOG.isDebugEnabled()) {
   LOG.debug("Pipeline creation successful. Pipeline: {}",
-  pipeline.toString());
+  pipeline);
 }
   }
 
diff --git 
a/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/HddsClientUtils.java
 
b/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/HddsClientUtils.java
index 8c9a11a..d7e3f4d 100644
--- 
a/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/HddsClientUtils.java
+++ 
b/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/HddsClientUtils.java
@@ -49,8 +49,6 @@ import org.apache.ratis.protocol.AlreadyClosedException;
 import org.apache.ratis.protocol.GroupMismatchException;
 import org.apache.ratis.protocol.NotReplicatedException;
 import org.apache.ratis.protocol.RaftRetryFailureException;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 import java.io.IOException;
 import java.net.InetSocketAddress;
@@ -76,9 +74,6 @@ import java.util.concurrent.TimeoutException;
 @InterfaceStability.Unstable
 public final class HddsClientUtils {
 
-  private static final Logger LOG = LoggerFactory.getLogger(
-  HddsClientUtils.class);
-
   private HddsClientUtils() {
   }
 
diff --git 
a/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/storage/BufferPool.java
 
b/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/storage/BufferPool.java
index f986fe3..f57700e 100644
--- 
a/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/storage/BufferPool.java
+++ 
b/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/storage/BufferPool.java
@@ -94,7 +94,7 @@ public class BufferPool {
 // always remove from head of the list and append at last
 final ChunkBuffer buffer = bufferList.remove(0);
 // Ensure the buffer to be r