Github user asfgit closed the pull request at:
https://github.com/apache/brooklyn-server/pull/285
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user ygy closed the pull request at:
https://github.com/apache/brooklyn-server/pull/285
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
GitHub user ygy reopened a pull request:
https://github.com/apache/brooklyn-server/pull/285
Adding HttpExecutor to be used for interchangeable implementations
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ygy/brooklyn-server
Github user ygy closed the pull request at:
https://github.com/apache/brooklyn-server/pull/285
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
GitHub user ygy reopened a pull request:
https://github.com/apache/brooklyn-server/pull/285
Adding HttpExecutor to be used for interchangeable implementations
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ygy/brooklyn-server
Github user ygy closed the pull request at:
https://github.com/apache/brooklyn-server/pull/285
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77546852
--- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java ---
@@ -407,14 +410,26 @@ private HttpToolResponse
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77537414
--- Diff:
core/src/main/java/org/apache/brooklyn/core/location/AbstractLocation.java ---
@@ -718,6 +719,35 @@ public boolean removeChild(Location
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77536131
--- Diff:
core/src/test/java/org/apache/brooklyn/util/http/executor/HttpExecutorImplTest.java
---
@@ -0,0 +1,210 @@
+/*
+ * Licensed to the
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77535728
--- Diff:
core/src/main/java/org/apache/brooklyn/util/http/executor/HttpExecutorFactoryImpl.java
---
@@ -0,0 +1,73 @@
+/*
+ * Licensed to
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77535388
--- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java ---
@@ -331,52 +366,55 @@ protected void preStart() {
}
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77535007
--- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java ---
@@ -331,52 +366,55 @@ protected void preStart() {
}
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77512748
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/executor/apacheclient/HttpResponseWrapper.java
---
@@ -0,0 +1,109 @@
+/*
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77511829
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/executor/apacheclient/HttpExecutorImpl.java
---
@@ -0,0 +1,113 @@
+/*
+ *
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77511724
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/executor/apacheclient/HttpExecutorImpl.java
---
@@ -0,0 +1,106 @@
+/*
+ *
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77509799
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/executor/apacheclient/HttpExecutorImpl.java
---
@@ -0,0 +1,113 @@
+/*
+ *
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77506454
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/executor/HttpRequest.java
---
@@ -0,0 +1,130 @@
+/*
+ * Licensed to the
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77501984
--- Diff:
core/src/main/java/org/apache/brooklyn/util/http/executor/HttpExecutorFactoryImpl.java
---
@@ -0,0 +1,75 @@
+/*
+ * Licensed to
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77501296
--- Diff:
core/src/main/java/org/apache/brooklyn/util/http/executor/HttpExecutorFactoryImpl.java
---
@@ -0,0 +1,75 @@
+/*
+ * Licensed to
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77500994
--- Diff:
core/src/main/java/org/apache/brooklyn/util/http/executor/HttpExecutorFactory.java
---
@@ -0,0 +1,29 @@
+/*
+ * Licensed to the
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77500841
--- Diff:
core/src/main/java/org/apache/brooklyn/location/byon/ByonLocationResolver.java
---
@@ -108,6 +109,7 @@ protected ConfigBag
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77500324
--- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java ---
@@ -331,52 +366,62 @@ protected void preStart() {
}
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77499318
--- Diff:
core/src/main/java/org/apache/brooklyn/core/location/AbstractLocation.java ---
@@ -718,6 +719,35 @@ public boolean removeChild(Location
Github user bostko commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77455765
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/executor/HttpRequest.java
---
@@ -0,0 +1,130 @@
+/*
+ * Licensed to the
GitHub user ygy reopened a pull request:
https://github.com/apache/brooklyn-server/pull/285
Adding HttpExecutor to be used for interchangeable implementations
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ygy/brooklyn-server
Github user ygy closed the pull request at:
https://github.com/apache/brooklyn-server/pull/285
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user ygy commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77386547
--- Diff:
core/src/main/java/org/apache/brooklyn/location/byon/ByonLocationResolver.java
---
@@ -108,6 +109,7 @@ protected ConfigBag extractConfig(Map
Github user ygy commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77386246
--- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java ---
@@ -331,52 +365,54 @@ protected void preStart() {
}
Github user ygy commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77385034
--- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java ---
@@ -275,7 +294,23 @@ public HttpFeed() {
protected HttpFeed(Builder
Github user ygy commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77378577
--- Diff:
core/src/test/java/org/apache/brooklyn/feed/http/HttpFeedTest.java ---
@@ -71,12 +71,12 @@
private static final long TIMEOUT_MS
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77348884
--- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java ---
@@ -331,52 +365,54 @@ protected void preStart() {
}
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77350751
--- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java ---
@@ -275,7 +294,23 @@ public HttpFeed() {
protected
Github user bostko commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r77345487
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/HttpTool.java ---
@@ -250,6 +254,21 @@ public static HttpClientBuilder
Github user ygy commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r76594668
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/executor/apacheclient/HttpExecutorImpl.java
---
@@ -0,0 +1,106 @@
+/*
+ *
Github user ygy commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r76594227
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/executor/HttpRequest.java
---
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the
Github user bostko commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r76591742
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/executor/HttpRequest.java
---
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the
Github user bostko commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r76590901
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/HttpTool.java ---
@@ -250,6 +254,21 @@ public static HttpClientBuilder
Github user bostko commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r76590580
--- Diff:
core/src/test/java/org/apache/brooklyn/feed/http/HttpFeedTest.java ---
@@ -71,12 +71,12 @@
private static final long
Github user bostko commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r76582992
--- Diff:
core/src/main/java/org/apache/brooklyn/util/http/executor/HttpExecutorFactoryImpl.java
---
@@ -0,0 +1,75 @@
+/*
+ * Licensed to
Github user aledsage commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r76570589
--- Diff:
core/src/main/java/org/apache/brooklyn/core/location/AbstractLocation.java ---
@@ -718,6 +719,35 @@ public boolean removeChild(Location
Github user iyovcheva commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r76260732
--- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java ---
@@ -331,52 +365,45 @@ protected void preStart() {
}
Github user bostko commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r75928906
--- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java ---
@@ -222,15 +238,17 @@ protected void finalize() {
}
Github user bostko commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r75928110
--- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java ---
@@ -331,52 +366,45 @@ protected void preStart() {
}
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r75689880
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/executor/apacheclient/HttpExecutorImpl.java
---
@@ -0,0 +1,106 @@
+/*
+ *
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r75689508
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/HttpExecutorFactory.java
---
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r75688643
--- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java ---
@@ -275,7 +293,23 @@ public HttpFeed() {
protected
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r75687462
--- Diff: core/src/main/java/org/apache/brooklyn/feed/http/HttpFeed.java ---
@@ -275,7 +293,23 @@ public HttpFeed() {
protected
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r75687134
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/executor/HttpRequest.java
---
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r75685586
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/executor/HttpRequest.java
---
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r75685265
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/executor/HttpRequest.java
---
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r75684207
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/HttpExecutorFactory.java
---
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r75683850
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/HttpExecutorFactory.java
---
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the
Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/285#discussion_r75683600
--- Diff:
utils/common/src/main/java/org/apache/brooklyn/util/http/HttpExecutorFactory.java
---
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the
53 matches
Mail list logo