Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/1785
---
Github user ppoulosk commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r108951566
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/utils/ArtifactoryConfigLoader.java
---
@@ -0,0 +1,429 @@
+/**
+ * Licensed to the Apache
Github user ppoulosk commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r108951633
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/utils/ArtifactoryConfigLoader.java
---
@@ -0,0 +1,429 @@
+/**
+ * Licensed to the Apache
Github user ppoulosk commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r108951217
--- Diff: docs/IConfigLoader.md ---
@@ -0,0 +1,58 @@
+---
+title: IConfigLoader
+layout: documentation
+documentation: true
+---
+
Github user ppoulosk commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r108951166
--- Diff: pom.xml ---
@@ -220,6 +220,7 @@
0.3.1
1.1.9
0.3.6
+1.2
--- End diff --
Fixing
Github user abellina commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r94978694
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/utils/ArtifactoryConfigLoader.java
---
@@ -0,0 +1,429 @@
+/**
+ * Licensed to the Apache
Github user abellina commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r94978032
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/resource/ResourceAwareScheduler.java
---
@@ -376,18 +388,12 @@ private void initialize(Topologies
Github user abellina commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r94978666
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/utils/ArtifactoryConfigLoader.java
---
@@ -0,0 +1,429 @@
+/**
+ * Licensed to the Apache
Github user abellina commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r94975625
--- Diff: pom.xml ---
@@ -220,6 +220,7 @@
0.3.1
1.1.9
0.3.6
+1.2
--- End diff --
why are we
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r94972777
--- Diff: docs/IConfigLoader.md ---
@@ -0,0 +1,58 @@
+---
+title: IConfigLoader
+layout: documentation
+documentation: true
+---
+
Github user jerrypeng commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r90526318
--- Diff: storm-core/src/jvm/org/apache/storm/Config.java ---
@@ -2177,13 +2177,37 @@
@isMapEntryType(keyType = String.class, valueType =
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r88932885
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/utils/ArtifactoryConfigLoader.java
---
@@ -0,0 +1,388 @@
+/**
+ * Licensed to the Apache
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r88935942
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/utils/SchedulerUtils.java ---
@@ -0,0 +1,64 @@
+/**
+ * Licensed to the Apache Software
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r88932491
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/multitenant/MultitenantScheduler.java
---
@@ -29,19 +29,35 @@
import
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r88936241
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/utils/SchedulerUtils.java ---
@@ -0,0 +1,64 @@
+/**
+ * Licensed to the Apache Software
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r88934621
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/utils/ArtifactoryConfigLoader.java
---
@@ -0,0 +1,388 @@
+/**
+ * Licensed to the Apache
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r88936089
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/utils/SchedulerUtils.java ---
@@ -0,0 +1,64 @@
+/**
+ * Licensed to the Apache Software
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r88932318
--- Diff: storm-core/src/jvm/org/apache/storm/Config.java ---
@@ -2177,13 +2177,37 @@
@isMapEntryType(keyType = String.class, valueType =
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r88935380
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/utils/IConfigLoader.java ---
@@ -0,0 +1,33 @@
+/**
+ * Licensed to the Apache Software
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r88935447
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/utils/IConfigLoader.java ---
@@ -0,0 +1,33 @@
+/**
+ * Licensed to the Apache Software
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r88931877
--- Diff: storm-core/src/jvm/org/apache/storm/Config.java ---
@@ -2177,13 +2177,37 @@
@isMapEntryType(keyType = String.class, valueType =
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r88934834
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/utils/ArtifactoryConfigLoader.java
---
@@ -0,0 +1,388 @@
+/**
+ * Licensed to the Apache
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1785#discussion_r88933434
--- Diff:
storm-core/src/jvm/org/apache/storm/scheduler/utils/ArtifactoryConfigLoader.java
---
@@ -0,0 +1,388 @@
+/**
+ * Licensed to the Apache
GitHub user ppoulosk opened a pull request:
https://github.com/apache/storm/pull/1785
[STORM-2201] Add dynamic scheduler configuration loading.
This adds an interface and two implementations, one that will load from a
local file, and another
that will load a config from
24 matches
Mail list logo