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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to