[GitHub] metron pull request #853: METRON-1337: List of facets should not be hardcode...

[merrimanr]

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

    https://github.com/apache/metron/pull/853#discussion_r169455383
  
    --- Diff: 
metron-interface/metron-rest/src/main/java/org/apache/metron/rest/service/impl/AlertServiceImpl.java
 ---
    @@ -37,15 +47,21 @@
     @Service
     public class AlertServiceImpl implements AlertService {
     
    --- End diff --
    
    Sure I can do that.  Since these settings are specific to the Alerts UI, 
should we include "ui" too?  Should the REST service also follow a similar 
pattern?


---