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. ---