You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
RAND_pseudo_bytes is deprecated in OpenSSL
1.1.1 and must be replaced by RAND_bytes.
Refactored usages of GetRandomClientName where
it now returns the client name wrapped in a
a shared_ptr to provide the ability to do
null check. We use a null check as a means to
get the functionality similar to monads in
functional programming.
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+1 💚
mvninstall
33m 30s
trunk passed
+1 💚
compile
2m 41s
trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚
compile
2m 42s
trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+1 💚
mvnsite
0m 31s
trunk passed
+1 💚
shadedclient
53m 7s
branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+1 💚
mvninstall
0m 15s
the patch passed
+1 💚
compile
2m 30s
the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+1 💚
mvninstall
33m 32s
trunk passed
+1 💚
compile
2m 43s
trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚
compile
2m 44s
trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+1 💚
mvnsite
0m 25s
trunk passed
+1 💚
shadedclient
53m 20s
branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+1 💚
mvninstall
0m 16s
the patch passed
+1 💚
compile
2m 36s
the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+1 💚
mvninstall
33m 37s
trunk passed
+1 💚
compile
2m 39s
trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚
compile
2m 45s
trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+1 💚
mvnsite
0m 29s
trunk passed
+1 💚
shadedclient
53m 15s
branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+1 💚
mvninstall
0m 15s
the patch passed
+1 💚
compile
2m 34s
the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
[WARNING] /home/jenkins/jenkins-agent/workspace/hadoop-multibranch_PR-2826/src/hadoop-hdfs-project/hadoop-hdfs-native-client/src/main/native/libhdfspp/lib/rpc/rpc_engine.cc:126:40: warning: 'int RAND_pseudo_bytes(unsigned char*, int)' is deprecated [-Wdeprecated-declarations]
[WARNING] from /home/jenkins/jenkins-agent/workspace/hadoop-multibranch_PR-2826/src/hadoop-hdfs-project/hadoop-hdfs-native-client/src/main/native/libhdfspp/lib/rpc/rpc_engine.cc:18:
[WARNING] /usr/include/openssl/rand.h:44:1: note: declared here
@goiri the above warnings are fixed in this PR - #2825. I created a separate PR for replacing each instance of RAND_pseudo_bytes since they involve quite a bit of refactoring and didn't want to complicate the PR review process.
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+1 💚
mvninstall
33m 19s
trunk passed
+1 💚
compile
2m 39s
trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚
compile
2m 52s
trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+1 💚
mvnsite
0m 29s
trunk passed
+1 💚
shadedclient
52m 54s
branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+1 💚
mvninstall
0m 16s
the patch passed
+1 💚
compile
2m 47s
the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
* RAND_pseudo_bytes is deprecated in OpenSSL
1.1.1 and must be replaced by RAND_bytes.
* Refactored usages of GetRandomClientName where
it now returns the client name wrapped in a
a shared_ptr to provide the ability to do
null check. We use a null check as a means to
get the functionality similar to monads in
functional programming.
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+1 💚
mvninstall
32m 20s
trunk passed
+1 💚
compile
2m 38s
trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚
compile
2m 44s
trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+1 💚
mvnsite
0m 29s
trunk passed
+1 💚
shadedclient
51m 37s
branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+1 💚
mvninstall
0m 16s
the patch passed
+1 💚
compile
2m 29s
the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚
cc
2m 29s
hadoop-hdfs-project_hadoop-hdfs-native-client-jdkUbuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 generated 0 new + 40 unchanged - 6 fixed = 40 total (was 46)
+1 💚
golang
2m 29s
the patch passed
+1 💚
javac
2m 29s
the patch passed
+1 💚
compile
2m 31s
the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+1 💚
cc
2m 31s
hadoop-hdfs-project_hadoop-hdfs-native-client-jdkPrivateBuild-1.8.0_282-8u282-b08-0ubuntu120.04-b08 with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu120.04-b08 generated 0 new + 40 unchanged - 6 fixed = 40 total (was 46)
+1 💚
golang
2m 31s
the patch passed
+1 💚
javac
2m 31s
the patch passed
+1 💚
blanks
0m 0s
The patch has no blanks issues.
+1 💚
mvnsite
0m 18s
the patch passed
+1 💚
shadedclient
13m 2s
patch has no errors when building and testing our client artifacts.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
1.1.1 and must be replaced by RAND_bytes.
it now returns the client name wrapped in a
a shared_ptr to provide the ability to do
null check. We use a null check as a means to
get the functionality similar to monads in
functional programming.