[ggj][engx] chore: remove debug printfs (#627)

* fix: fix dep ordering in Bazel dedupe rules

* feat: add a basic trie

* fix: add hashCode() and equals() to Field

* fix: add equals() and hashCode() for MethodArgument

* feat: add a TriFunction

* feat: add DFS traversal to Trie

* feat: add support for nested method sigs (aip.dev/4232)

* chore: remove debug printfs

Author: miraleung

src/main/java/com/google/api/generator/gapic/protoparser/HttpRuleParser.java

@@ -130,11 +130,8 @@ private static void checkHttpFieldIsValid(String binding, Message inputMessage,
     Preconditions.checkState(
         inputMessage.fieldMap().containsKey(binding),
         String.format(
-            "Expected message %s to contain field %s but none found"
-                + ", DEL: "
-                + inputMessage.fieldMap().keySet(),
-            inputMessage.name(),
-            binding));
+            "Expected message %s to contain field %s but none found",
+            inputMessage.name(), binding));
     Field field = inputMessage.fieldMap().get(binding);
     boolean fieldCondition = !field.isRepeated();
     if (!isBody) {

src/main/java/com/google/api/generator/gapic/protoparser/Parser.java

@@ -569,9 +569,8 @@ static LongrunningOperation parseLro(
     Preconditions.checkNotNull(
         metadataMessage,
         String.format(
-            "LRO metadata message %s not found in method %s" + ", DEL: " + messageTypes.keySet(),
-            metadataTypeName,
-            methodDescriptor.getName()));
+            "LRO metadata message %s not found in method %s",
+            metadataTypeName, methodDescriptor.getName()));
 
     return LongrunningOperation.withTypes(responseMessage.type(), metadataMessage.type());
   }