Zoltan Chovan has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22703 )

Change subject: Add SecurityManagerCompatibility shim
......................................................................


Patch Set 3:

(2 comments)

> Patch Set 3:
>
> Commit message changed, last time I also fixed some related test failures, 
> but this time I can not understand the issue

If you take a look at the jenkins output 
(https://jenkins.kudu.apache.org/job/build_and_test/7619/console) you can see 
that some Java tests have failed. To see which ones you can check the dist-test 
results: 
http://dist-test.cloudera.org/job?job_id=jenkins-slave.1743161364.2709138

It seems that org.apache.kudu.spark.kudu.DefaultSourceTest and 
org.apache.kudu.client.TestSecurity are both failing consistently across 
builds. You'll have to fix these test failures to get a green build from 
jenkins.

http://gerrit.cloudera.org:8080/#/c/22703/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/22703/1//COMMIT_MSG@7
PS1, Line 7: Add SecurityManagerCompatibility shim
Please add some details to the commit message about this change.


http://gerrit.cloudera.org:8080/#/c/22703/1/java/kudu-client/src/main/java/org/apache/kudu/client/internals/CompositeStrategy.java
File 
java/kudu-client/src/main/java/org/apache/kudu/client/internals/CompositeStrategy.java:

http://gerrit.cloudera.org:8080/#/c/22703/1/java/kudu-client/src/main/java/org/apache/kudu/client/internals/CompositeStrategy.java@48
PS1, Line 48:     SecurityManagerCompatibility fallback = null;
no need to init to null



--
To view, visit http://gerrit.cloudera.org:8080/22703
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I2a0f7306d9471c6a86bff3e4864947a0603337d7
Gerrit-Change-Number: 22703
Gerrit-PatchSet: 3
Gerrit-Owner: Vladyslav Lyutenko <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Vladyslav Lyutenko <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Comment-Date: Mon, 31 Mar 2025 08:14:28 +0000
Gerrit-HasComments: Yes

Reply via email to