removed some log output used for debugging

Author: jasonbrudvikess

app/save-and-restore/app/src/main/java/org/phoebus/applications/saveandrestore/ui/search/SearchAndFilterViewController.java

@@ -230,9 +230,7 @@ public void initialize(URL url, ResourceBundle resourceBundle) {
             if (e.getCode() == KeyCode.ENTER) {
                 LOGGER.log(Level.INFO, "ENTER has been pressed in uniqueIdTextField");
                 LOGGER.log(Level.INFO, "uniqueIdProperty: " + uniqueIdProperty.getValueSafe());
-                if (uniqueIdProperty.isEmpty().get()) {
-                    LOGGER.log(Level.INFO, "uniqueIdString: is empty");
-                } else {
+                if (!uniqueIdProperty.isEmpty().get()) {
                     searchResultTableViewController.uniqueIdSearch(uniqueIdProperty.getValueSafe());
                 }
             }

app/save-and-restore/app/src/main/java/org/phoebus/applications/saveandrestore/ui/search/SearchResultTableViewController.java

@@ -328,7 +328,6 @@ public void search() {
 
     public void search(final String query) {
         queryString = query;
-        LOGGER.log(Level.INFO, "search() query: = " + queryString);
         Map<String, String> searchParams =
                 SearchQueryUtil.parseHumanReadableQueryString(queryString);
         LOGGER.log(Level.INFO, "search() searchParams: = " + searchParams);
@@ -370,7 +369,6 @@ void uniqueIdSearch(final String uniqueIdString) {
         try {
             /* Search with the uniqueID */
             Node uniqueIdNode = SaveAndRestoreService.getInstance().getNode(uniqueIdString);
-            LOGGER.log(Level.INFO, "uniqueIDNode: " + uniqueIdNode);
 
             /* Check that there are results, then fill table - should be at most one result */
             if (uniqueIdNode != null) {

app/save-and-restore/model/src/main/java/org/phoebus/applications/saveandrestore/model/search/SearchQueryUtil.java

@@ -26,8 +26,6 @@
 import java.util.Map;
 import java.util.function.Function;
 import java.util.stream.Collectors;
-import java.util.logging.Logger;
-import java.util.logging.Level;
 
 public class SearchQueryUtil {
 
@@ -79,8 +77,6 @@ public static Keys findKey(String keyName) {
         }
     }
 
-    private static final Logger LOGGER = Logger.getLogger(SearchQueryUtil.class.getName());
-
     /**
      * This method parses a logbook query string and returns a map of search keys and their associated search patterns as
      * values.
@@ -124,8 +120,6 @@ public static String toQueryString(Map<String, String> queryParams){
      * @return A formatted string with trimmed values, e.g. "a,b" rather than " a , b".
      */
     private static String formatSearchTerm(String searchTerm){
-        LOGGER.log(Level.INFO, "formatSearchTerm() input:  [" + searchTerm + "]");
-
         if(searchTerm == null || searchTerm.isEmpty()){
             return "";
         }

services/save-and-restore/src/main/java/org/phoebus/service/saveandrestore/search/SearchUtil.java

@@ -75,7 +75,6 @@ public SearchRequest buildSearchRequest(MultiValueMap<String, String> searchPara
         LOG.info("  searchParameters: " + searchParameters);
 
         for (Entry<String, List<String>> parameter : searchParameters.entrySet()) {
-            LOG.info("    key: " + parameter.getKey().strip().toLowerCase());
             switch (parameter.getKey().strip().toLowerCase()) {
                 case "uniqueid":
                     for (String value : parameter.getValue()) {
@@ -84,14 +83,12 @@ public SearchRequest buildSearchRequest(MultiValueMap<String, String> searchPara
                         }
                     }
                     break;
+
                 // Search for node name. List of names cannot be split on space char as it is allowed in a node name.
                 case "name":
                     for (String value : parameter.getValue()) {
-                        LOG.info("    value: [" + value + "]");
                         for (String pattern : getSearchTerms(value)) {
-                        // for (String pattern : value.split("[|,;]")) {
                             String term = pattern.trim().toLowerCase();
-                            LOG.info("        term: [" + term + "]");
                             // Quoted strings will be mapped to a phrase query
                             if(term.startsWith("\"") && term.endsWith("\"")){
                                 nodeNamePhraseTerms.add(term.substring(1, term.length() - 1));
@@ -108,10 +105,8 @@ public SearchRequest buildSearchRequest(MultiValueMap<String, String> searchPara
                 case "desc":
                 case "description":
                     for (String value : parameter.getValue()) {
-                        LOG.info("    value: [" + value + "]");
                         for (String pattern : getSearchTerms(value)) {
                             String term = pattern.trim().toLowerCase();
-                            LOG.info("        term: [" + term + "]");
                             // Quoted strings will be mapped to a phrase query
                             if (term.startsWith("\"") && term.endsWith("\"")) {
                                 descriptionPhraseTerms.add(term.substring(1, term.length() - 1));
@@ -240,11 +235,8 @@ public SearchRequest buildSearchRequest(MultiValueMap<String, String> searchPara
             }
         }
 
-        LOG.info("  descriptionTerms: " + descriptionTerms);
-        LOG.info("  descriptionTerms.isEmpty() : " + descriptionTerms.isEmpty());
         // Add the description query. Multiple search terms will be AND:ed.
         if (!descriptionTerms.isEmpty()) {
-            LOG.info("  fuzzySearch: " + fuzzySearch);
             for (String searchTerm : descriptionTerms) {
                 NestedQuery innerNestedQuery;
                 if (fuzzySearch) {
@@ -258,8 +250,6 @@ public SearchRequest buildSearchRequest(MultiValueMap<String, String> searchPara
             }
         }
 
-        LOG.info("  descriptionPhraseTerms: " + descriptionPhraseTerms);
-        LOG.info("  descriptionPhraseTerms.isEmpty() : " + descriptionPhraseTerms.isEmpty());
         // Add phrase queries for the description key. Multiple search terms will be AND:ed.
         if (!descriptionPhraseTerms.isEmpty()) {
             for (String searchTerm : descriptionPhraseTerms) {
@@ -274,11 +264,8 @@ public SearchRequest buildSearchRequest(MultiValueMap<String, String> searchPara
             boolQueryBuilder.must(IdsQuery.of(id -> id.values(uniqueIdTerms))._toQuery());
         }
 
-        LOG.info("  nodeNameTerms: " + nodeNameTerms);
-        LOG.info("  nodeNameTerms.isEmpty() : " + nodeNameTerms.isEmpty());
         // Add the description query. Multiple search terms will be AND:ed.
         if (!nodeNameTerms.isEmpty()) {
-            LOG.info("  fuzzySearch: " + fuzzySearch);
             for (String searchTerm : nodeNameTerms) {
                 NestedQuery innerNestedQuery;
                 if (fuzzySearch) {
@@ -292,8 +279,6 @@ public SearchRequest buildSearchRequest(MultiValueMap<String, String> searchPara
             }
         }
 
-        LOG.info("  nodeNamePhraseTerms: " + nodeNamePhraseTerms);
-        LOG.info("  nodeNamePhraseTerms.isEmpty() : " + nodeNamePhraseTerms.isEmpty());
         // Add phrase queries for the nodeName key. Multiple search terms will be AND:ed.
         if (!nodeNamePhraseTerms.isEmpty()) {
             for (String searchTerm : nodeNamePhraseTerms) {