homatthew commented on code in PR #3678:
URL: https://github.com/apache/gobblin/pull/3678#discussion_r1164794522


##########
gobblin-yarn/src/main/java/org/apache/gobblin/yarn/YarnService.java:
##########
@@ -453,7 +453,7 @@ private EventSubmitter buildEventSubmitter() {
   /**
    * Request an allocation of containers. If numTargetContainers is larger 
than the max of current and expected number
    * of containers then additional containers are requested.
-   *
+   * <p>

Review Comment:
   Dumb question, but does <p> need a closing tag? The placement of these tags 
make it seem like you meant to do <br> 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to