[GitHub] [gobblin] homatthew commented on a diff in pull request #3678: [GOBBLIN-1817] change some deprecated code and fix minor codestyle

["homatthew (via GitHub)" <gi...@apache.org>]

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: dev-unsubscribe@gobblin.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org