hadoop git commit: HADOOP-10829. Iteration on CredentialProviderFactory.serviceLoader is thread-unsafe. Contributed by Benoy Antony and Rakesh R.

2017-08-31 Thread junping_du
Repository: hadoop
Updated Branches:
  refs/heads/branch-2.8.2 8e0e71075 -> a80cd4b89


HADOOP-10829. Iteration on CredentialProviderFactory.serviceLoader is 
thread-unsafe. Contributed by Benoy Antony and Rakesh R.

(cherry picked from commit b82485d6fed2194bf0dc2eedfab7e226e30a7cf0)
(cherry picked from commit c0e294ffc82d611a64b352e3962bce9399ddbe5f)


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/a80cd4b8
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/a80cd4b8
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/a80cd4b8

Branch: refs/heads/branch-2.8.2
Commit: a80cd4b897216b8aa2bfd2f4fb5c37a545ef4503
Parents: 8e0e7107
Author: Jitendra Pandey 
Authored: Fri Jul 7 12:45:37 2017 -0700
Committer: Junping Du 
Committed: Thu Aug 31 16:23:37 2017 -0700

--
 .../hadoop/security/alias/CredentialProviderFactory.java  | 10 ++
 1 file changed, 10 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/hadoop/blob/a80cd4b8/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
--
diff --git 
a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
 
b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
index 3bb4f6d..1f5c8ec 100644
--- 
a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
+++ 
b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
@@ -22,6 +22,7 @@ import java.io.IOException;
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.util.ArrayList;
+import java.util.Iterator;
 import java.util.List;
 import java.util.ServiceLoader;
 
@@ -47,6 +48,15 @@ public abstract class CredentialProviderFactory {
   private static final ServiceLoader serviceLoader =
   ServiceLoader.load(CredentialProviderFactory.class);
 
+  // Iterate through the serviceLoader to avoid lazy loading.
+  // Lazy loading would require synchronization in concurrent use cases.
+  static {
+Iterator iterServices = 
serviceLoader.iterator();
+while (iterServices.hasNext()) {
+  iterServices.next();
+}
+  }
+
   public static List getProviders(Configuration conf
) throws IOException {
 List result = new ArrayList();


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



hadoop git commit: HADOOP-10829. Iteration on CredentialProviderFactory.serviceLoader is thread-unsafe. Contributed by Benoy Antony and Rakesh R.

2017-07-25 Thread zhz
Repository: hadoop
Updated Branches:
  refs/heads/branch-2.7 62ce8f043 -> 18fe88ac6


HADOOP-10829. Iteration on CredentialProviderFactory.serviceLoader is 
thread-unsafe. Contributed by Benoy Antony and Rakesh R.

(cherry picked from commit b82485d6fed2194bf0dc2eedfab7e226e30a7cf0)
(cherry picked from commit c0e294ffc82d611a64b352e3962bce9399ddbe5f)


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/18fe88ac
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/18fe88ac
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/18fe88ac

Branch: refs/heads/branch-2.7
Commit: 18fe88ac63d53cab474d101616b4df64b40f8a56
Parents: 62ce8f0
Author: Jitendra Pandey 
Authored: Fri Jul 7 12:45:37 2017 -0700
Committer: Zhe Zhang 
Committed: Tue Jul 25 09:07:40 2017 -0700

--
 .../hadoop/security/alias/CredentialProviderFactory.java  | 10 ++
 1 file changed, 10 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/hadoop/blob/18fe88ac/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
--
diff --git 
a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
 
b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
index 3bb4f6d..1f5c8ec 100644
--- 
a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
+++ 
b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
@@ -22,6 +22,7 @@ import java.io.IOException;
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.util.ArrayList;
+import java.util.Iterator;
 import java.util.List;
 import java.util.ServiceLoader;
 
@@ -47,6 +48,15 @@ public abstract class CredentialProviderFactory {
   private static final ServiceLoader serviceLoader =
   ServiceLoader.load(CredentialProviderFactory.class);
 
+  // Iterate through the serviceLoader to avoid lazy loading.
+  // Lazy loading would require synchronization in concurrent use cases.
+  static {
+Iterator iterServices = 
serviceLoader.iterator();
+while (iterServices.hasNext()) {
+  iterServices.next();
+}
+  }
+
   public static List getProviders(Configuration conf
) throws IOException {
 List result = new ArrayList();


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



hadoop git commit: HADOOP-10829. Iteration on CredentialProviderFactory.serviceLoader is thread-unsafe. Contributed by Benoy Antony and Rakesh R.

2017-07-25 Thread zhz
Repository: hadoop
Updated Branches:
  refs/heads/branch-2.8 7be52c9c7 -> c0e294ffc


HADOOP-10829. Iteration on CredentialProviderFactory.serviceLoader is 
thread-unsafe. Contributed by Benoy Antony and Rakesh R.

(cherry picked from commit b82485d6fed2194bf0dc2eedfab7e226e30a7cf0)


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/c0e294ff
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/c0e294ff
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/c0e294ff

Branch: refs/heads/branch-2.8
Commit: c0e294ffc82d611a64b352e3962bce9399ddbe5f
Parents: 7be52c9
Author: Jitendra Pandey 
Authored: Fri Jul 7 12:45:37 2017 -0700
Committer: Zhe Zhang 
Committed: Tue Jul 25 09:04:49 2017 -0700

--
 .../hadoop/security/alias/CredentialProviderFactory.java  | 10 ++
 1 file changed, 10 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/hadoop/blob/c0e294ff/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
--
diff --git 
a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
 
b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
index 3bb4f6d..1f5c8ec 100644
--- 
a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
+++ 
b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
@@ -22,6 +22,7 @@ import java.io.IOException;
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.util.ArrayList;
+import java.util.Iterator;
 import java.util.List;
 import java.util.ServiceLoader;
 
@@ -47,6 +48,15 @@ public abstract class CredentialProviderFactory {
   private static final ServiceLoader serviceLoader =
   ServiceLoader.load(CredentialProviderFactory.class);
 
+  // Iterate through the serviceLoader to avoid lazy loading.
+  // Lazy loading would require synchronization in concurrent use cases.
+  static {
+Iterator iterServices = 
serviceLoader.iterator();
+while (iterServices.hasNext()) {
+  iterServices.next();
+}
+  }
+
   public static List getProviders(Configuration conf
) throws IOException {
 List result = new ArrayList();


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



[41/50] [abbrv] hadoop git commit: HADOOP-10829. Iteration on CredentialProviderFactory.serviceLoader is thread-unsafe. Contributed by Benoy Antony and Rakesh R.

2017-07-11 Thread asuresh
HADOOP-10829. Iteration on CredentialProviderFactory.serviceLoader is 
thread-unsafe. Contributed by Benoy Antony and Rakesh R.


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/f1efa14f
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/f1efa14f
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/f1efa14f

Branch: refs/heads/YARN-5972
Commit: f1efa14fc676641fa15c11d3147e3ad948b084e9
Parents: 5496a34
Author: Jitendra Pandey 
Authored: Fri Jul 7 12:45:37 2017 -0700
Committer: Jitendra Pandey 
Committed: Mon Jul 10 17:48:27 2017 -0700

--
 .../hadoop/security/alias/CredentialProviderFactory.java  | 10 ++
 1 file changed, 10 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/hadoop/blob/f1efa14f/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
--
diff --git 
a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
 
b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
index d1e3eb5..1b2ac41 100644
--- 
a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
+++ 
b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
@@ -22,6 +22,7 @@ import java.io.IOException;
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.util.ArrayList;
+import java.util.Iterator;
 import java.util.List;
 import java.util.ServiceLoader;
 
@@ -49,6 +50,15 @@ public abstract class CredentialProviderFactory {
   ServiceLoader.load(CredentialProviderFactory.class,
   CredentialProviderFactory.class.getClassLoader());
 
+  // Iterate through the serviceLoader to avoid lazy loading.
+  // Lazy loading would require synchronization in concurrent use cases.
+  static {
+Iterator iterServices = 
serviceLoader.iterator();
+while (iterServices.hasNext()) {
+  iterServices.next();
+}
+  }
+
   public static List getProviders(Configuration conf
) throws IOException {
 List result = new ArrayList();


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



[16/50] [abbrv] hadoop git commit: HADOOP-10829. Iteration on CredentialProviderFactory.serviceLoader is thread-unsafe. Contributed by Benoy Antony and Rakesh R.

2017-07-11 Thread rakeshr
HADOOP-10829. Iteration on CredentialProviderFactory.serviceLoader is 
thread-unsafe. Contributed by Benoy Antony and Rakesh R.


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/f1efa14f
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/f1efa14f
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/f1efa14f

Branch: refs/heads/HDFS-10285
Commit: f1efa14fc676641fa15c11d3147e3ad948b084e9
Parents: 5496a34
Author: Jitendra Pandey 
Authored: Fri Jul 7 12:45:37 2017 -0700
Committer: Jitendra Pandey 
Committed: Mon Jul 10 17:48:27 2017 -0700

--
 .../hadoop/security/alias/CredentialProviderFactory.java  | 10 ++
 1 file changed, 10 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/hadoop/blob/f1efa14f/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
--
diff --git 
a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
 
b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
index d1e3eb5..1b2ac41 100644
--- 
a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
+++ 
b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
@@ -22,6 +22,7 @@ import java.io.IOException;
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.util.ArrayList;
+import java.util.Iterator;
 import java.util.List;
 import java.util.ServiceLoader;
 
@@ -49,6 +50,15 @@ public abstract class CredentialProviderFactory {
   ServiceLoader.load(CredentialProviderFactory.class,
   CredentialProviderFactory.class.getClassLoader());
 
+  // Iterate through the serviceLoader to avoid lazy loading.
+  // Lazy loading would require synchronization in concurrent use cases.
+  static {
+Iterator iterServices = 
serviceLoader.iterator();
+while (iterServices.hasNext()) {
+  iterServices.next();
+}
+  }
+
   public static List getProviders(Configuration conf
) throws IOException {
 List result = new ArrayList();


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



hadoop git commit: HADOOP-10829. Iteration on CredentialProviderFactory.serviceLoader is thread-unsafe. Contributed by Benoy Antony and Rakesh R.

2017-07-10 Thread jitendra
Repository: hadoop
Updated Branches:
  refs/heads/branch-2 d94b30cb0 -> b82485d6f


HADOOP-10829. Iteration on CredentialProviderFactory.serviceLoader is 
thread-unsafe. Contributed by Benoy Antony and Rakesh R.


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/b82485d6
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/b82485d6
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/b82485d6

Branch: refs/heads/branch-2
Commit: b82485d6fed2194bf0dc2eedfab7e226e30a7cf0
Parents: d94b30c
Author: Jitendra Pandey 
Authored: Fri Jul 7 12:45:37 2017 -0700
Committer: Jitendra Pandey 
Committed: Mon Jul 10 17:55:02 2017 -0700

--
 .../hadoop/security/alias/CredentialProviderFactory.java  | 10 ++
 1 file changed, 10 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/hadoop/blob/b82485d6/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
--
diff --git 
a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
 
b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
index d1e3eb5..1b2ac41 100644
--- 
a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
+++ 
b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
@@ -22,6 +22,7 @@ import java.io.IOException;
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.util.ArrayList;
+import java.util.Iterator;
 import java.util.List;
 import java.util.ServiceLoader;
 
@@ -49,6 +50,15 @@ public abstract class CredentialProviderFactory {
   ServiceLoader.load(CredentialProviderFactory.class,
   CredentialProviderFactory.class.getClassLoader());
 
+  // Iterate through the serviceLoader to avoid lazy loading.
+  // Lazy loading would require synchronization in concurrent use cases.
+  static {
+Iterator iterServices = 
serviceLoader.iterator();
+while (iterServices.hasNext()) {
+  iterServices.next();
+}
+  }
+
   public static List getProviders(Configuration conf
) throws IOException {
 List result = new ArrayList();


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



hadoop git commit: HADOOP-10829. Iteration on CredentialProviderFactory.serviceLoader is thread-unsafe. Contributed by Benoy Antony and Rakesh R.

2017-07-10 Thread jitendra
Repository: hadoop
Updated Branches:
  refs/heads/trunk 5496a34c0 -> f1efa14fc


HADOOP-10829. Iteration on CredentialProviderFactory.serviceLoader is 
thread-unsafe. Contributed by Benoy Antony and Rakesh R.


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/f1efa14f
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/f1efa14f
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/f1efa14f

Branch: refs/heads/trunk
Commit: f1efa14fc676641fa15c11d3147e3ad948b084e9
Parents: 5496a34
Author: Jitendra Pandey 
Authored: Fri Jul 7 12:45:37 2017 -0700
Committer: Jitendra Pandey 
Committed: Mon Jul 10 17:48:27 2017 -0700

--
 .../hadoop/security/alias/CredentialProviderFactory.java  | 10 ++
 1 file changed, 10 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/hadoop/blob/f1efa14f/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
--
diff --git 
a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
 
b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
index d1e3eb5..1b2ac41 100644
--- 
a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
+++ 
b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
@@ -22,6 +22,7 @@ import java.io.IOException;
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.util.ArrayList;
+import java.util.Iterator;
 import java.util.List;
 import java.util.ServiceLoader;
 
@@ -49,6 +50,15 @@ public abstract class CredentialProviderFactory {
   ServiceLoader.load(CredentialProviderFactory.class,
   CredentialProviderFactory.class.getClassLoader());
 
+  // Iterate through the serviceLoader to avoid lazy loading.
+  // Lazy loading would require synchronization in concurrent use cases.
+  static {
+Iterator iterServices = 
serviceLoader.iterator();
+while (iterServices.hasNext()) {
+  iterServices.next();
+}
+  }
+
   public static List getProviders(Configuration conf
) throws IOException {
 List result = new ArrayList();


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