You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by ma...@apache.org on 2016/08/19 18:34:04 UTC
incubator-ranger git commit: RANGER-1155: potential
performance/correctness issues uncovered by static code analysis
Repository: incubator-ranger
Updated Branches:
refs/heads/master 89136413e -> bf3f09542
RANGER-1155: potential performance/correctness issues uncovered by static code analysis
Signed-off-by: Madhan Neethiraj <ma...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/incubator-ranger/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-ranger/commit/bf3f0954
Tree: http://git-wip-us.apache.org/repos/asf/incubator-ranger/tree/bf3f0954
Diff: http://git-wip-us.apache.org/repos/asf/incubator-ranger/diff/bf3f0954
Branch: refs/heads/master
Commit: bf3f095429b0027213c6e10dd6a32d94608623d3
Parents: 8913641
Author: Abhay Kulkarni <ak...@hortonworks.com>
Authored: Thu Aug 18 16:15:38 2016 -0700
Committer: Madhan Neethiraj <ma...@apache.org>
Committed: Fri Aug 19 10:44:21 2016 -0700
----------------------------------------------------------------------
.../ranger/plugin/policyengine/RangerPolicyRepository.java | 2 +-
.../org/apache/ranger/plugin/service/RangerBasePlugin.java | 2 +-
.../ranger/policyengine/RangerPolicyenginePerfTester.java | 8 ++++++--
3 files changed, 8 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-ranger/blob/bf3f0954/agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java
----------------------------------------------------------------------
diff --git a/agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java b/agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java
index 0012329..40fe8b6 100644
--- a/agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java
+++ b/agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java
@@ -51,7 +51,7 @@ class RangerPolicyRepository {
AUDIT_ALL, AUDIT_NONE, AUDIT_DEFAULT
}
- class AuditInfo {
+ static private final class AuditInfo {
final boolean isAudited;
final long auditPolicyId;
http://git-wip-us.apache.org/repos/asf/incubator-ranger/blob/bf3f0954/agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java
----------------------------------------------------------------------
diff --git a/agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java b/agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java
index 760c4a3..760fab7 100644
--- a/agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java
+++ b/agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java
@@ -447,7 +447,7 @@ public class RangerBasePlugin {
int counter=0;
}
- private class PolicyEngineRefresher extends TimerTask {
+ static private final class PolicyEngineRefresher extends TimerTask {
private final RangerBasePlugin plugin;
PolicyEngineRefresher(RangerBasePlugin plugin) {
http://git-wip-us.apache.org/repos/asf/incubator-ranger/blob/bf3f0954/ranger-tools/src/main/java/org/apache/ranger/policyengine/RangerPolicyenginePerfTester.java
----------------------------------------------------------------------
diff --git a/ranger-tools/src/main/java/org/apache/ranger/policyengine/RangerPolicyenginePerfTester.java b/ranger-tools/src/main/java/org/apache/ranger/policyengine/RangerPolicyenginePerfTester.java
index 50432f7..03ea6d0 100644
--- a/ranger-tools/src/main/java/org/apache/ranger/policyengine/RangerPolicyenginePerfTester.java
+++ b/ranger-tools/src/main/java/org/apache/ranger/policyengine/RangerPolicyenginePerfTester.java
@@ -69,8 +69,12 @@ public class RangerPolicyenginePerfTester {
for(URL requestFileURL : requestFileURLs) {
PerfTestClient perfTestClient = new PerfTestClient(perfTestEngine, 0, requestFileURL, 1);
- perfTestClient.init();
- perfTestClient.run();
+ if (perfTestClient.init()) {
+ perfTestClient.start();
+ perfTestClient.join();
+ } else {
+ LOG.error("Error initializing warm-up PerfTestClient");
+ }
}
} catch(Throwable t) {
LOG.error("Error during warmup", t);