Skip to content

Commit

Permalink
[ggj][engx] chore: remove debug printfs (#627)
Browse files Browse the repository at this point in the history
* 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
  • Loading branch information
miraleung committed Dec 24, 2020
1 parent 60f65ea commit 767874f
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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());
}
Expand Down

0 comments on commit 767874f

Please sign in to comment.