metron-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From iraghumitra <...@git.apache.org>
Subject [GitHub] metron issue #620: Metron-988: UI for viewing alerts generated by Metron
Date Sun, 02 Jul 2017 20:46:58 GMT
Github user iraghumitra commented on the issue:

    https://github.com/apache/metron/pull/620
  
    @merrimanr QueryBuilder is thought of as a model than a utils class, the fields with '_'
are used for display purpose and are not required to be persisted. I will take an another
pass to see if I can refactor the code to make it look simple.In fact, the Saved Search and
Recent Search Saves the query builder.
    I understand that SearchRequest might be a more suitable name so renaming QueryBuilder
to SearchRequest makes sense ?. I am taking an another pass at abstracting all the API calls
so that we can change the api provider to 'solr' or rest api easily form the UI. Once i do
that the service classes would look better.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message