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

    https://github.com/apache/metron/pull/1047#discussion_r194168449
  
    --- Diff: 
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/configuration/metron-env.xml
 ---
    @@ -137,4 +137,10 @@
             <value>yyyy.MM.dd.HH</value>
             <display-name>Elasticsearch Date Format</display-name>
         </property>
    +    <property>
    --- End diff --
    
    Do you think It might be more appropriate to have this property in 
metron-rest-env instead?  Now when I change it I'm prompted to restart all 
Metron components when really the only one that needs to be restarted is REST 
and the UIs.  Both UIs depend on REST so they will also be included in the 
restart list.


---

Reply via email to