[GitHub] storm pull request #2806: [STORM-3198] Topology submitters should be able to...

2018-09-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/2806 ---

[GitHub] storm pull request #2806: [STORM-3198] Topology submitters should be able to...

2018-08-20 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2806#discussion_r211484225 --- Diff: storm-server/src/main/java/org/apache/storm/daemon/supervisor/BasicContainer.java --- @@ -622,6 +620,12 @@ protected String javaCmd(String cmd) {

[GitHub] storm pull request #2806: [STORM-3198] Topology submitters should be able to...

2018-08-16 Thread jacobtolar
Github user jacobtolar commented on a diff in the pull request: https://github.com/apache/storm/pull/2806#discussion_r210769170 --- Diff: pom.xml --- @@ -278,7 +278,7 @@ 4.1.25.Final 1.0.2 1.6.6 -2.8.2 +2.11.0 ---

[GitHub] storm pull request #2806: [STORM-3198] Topology submitters should be able to...

2018-08-16 Thread jacobtolar
Github user jacobtolar commented on a diff in the pull request: https://github.com/apache/storm/pull/2806#discussion_r210768599 --- Diff: storm-server/src/main/java/org/apache/storm/daemon/supervisor/BasicContainer.java --- @@ -622,6 +620,12 @@ protected String javaCmd(String

[GitHub] storm pull request #2806: [STORM-3198] Topology submitters should be able to...

2018-08-16 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2806#discussion_r210681943 --- Diff: storm-server/src/main/java/org/apache/storm/daemon/supervisor/BasicContainer.java --- @@ -622,6 +620,12 @@ protected String javaCmd(String cmd) {

[GitHub] storm pull request #2806: [STORM-3198] Topology submitters should be able to...

2018-08-16 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2806#discussion_r210683327 --- Diff: pom.xml --- @@ -278,7 +278,7 @@ 4.1.25.Final 1.0.2 1.6.6 -2.8.2 +2.11.0 --- End

[GitHub] storm pull request #2806: [STORM-3198] Topology submitters should be able to...

2018-08-16 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2806#discussion_r210682266 --- Diff: storm-client/src/jvm/org/apache/storm/Config.java --- @@ -705,6 +705,13 @@ */ @isString(acceptedValues = { "S0", "S1", "S2", "S3"

[GitHub] storm pull request #2806: [STORM-3198] Topology submitters should be able to...

2018-08-16 Thread jacobtolar
Github user jacobtolar commented on a diff in the pull request: https://github.com/apache/storm/pull/2806#discussion_r210671831 --- Diff: pom.xml --- @@ -278,7 +278,7 @@ 4.1.25.Final 1.0.2 1.6.6 -2.8.2 +2.11.0 ---

[GitHub] storm pull request #2806: [STORM-3198] Topology submitters should be able to...

2018-08-16 Thread jacobtolar
GitHub user jacobtolar opened a pull request: https://github.com/apache/storm/pull/2806 [STORM-3198] Topology submitters should be able to supply log4j2 conf This adds a new config setting, `topology.logging.config`, that allows a topology submitter to specify an additional log4j2