Github user nickwallen commented on a diff in the pull request:

    https://github.com/apache/metron/pull/1211#discussion_r223501886
  
    --- Diff: 
metron-analytics/metron-profiler-storm/src/test/java/org/apache/metron/profiler/storm/integration/ConfigUploadComponent.java
 ---
    @@ -117,8 +127,13 @@ public ConfigUploadComponent 
withGlobalConfiguration(String path) {
         return this;
       }
     
    -  public ConfigUploadComponent withProfilerConfiguration(String path) {
    -    this.profilerConfiguration = path;
    +  public ConfigUploadComponent withProfilerConfigurationPath(String path) {
    --- End diff --
    
    I needed this to let me upload the profile definitions, which for the 
integration tests, are just Strings.


---

Reply via email to