[ https://issues.apache.org/jira/browse/YARN-7072?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16158305#comment-16158305 ]
Junping Du commented on YARN-7072: ---------------------------------- Patch LGTM in overall. The findbug warning is not related. However two javadoc errors should get fixed: {code} ... [ERROR] /testptch/hadoop/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/logaggregation/ContainerLogMeta.java:29: error: reference not found [ERROR] * <li>A list of {@link PerContainerLogFileInfo}.</li> [ERROR] ^ ... [ERROR] /testptch/hadoop/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/main/java/org/apache/hadoop/yarn/server/webapp/dao/ContainerLogsInfo.java:37: error: reference not found [ERROR] * <li>A list of {@link PerContainerLogFileInfo}.</li> [ERROR] ^ {code} Will +1 after javadoc warnings get fixed. Some NITs could get fixed later in other following JIRAs. > Add a new log aggregation file format controller > ------------------------------------------------ > > Key: YARN-7072 > URL: https://issues.apache.org/jira/browse/YARN-7072 > Project: Hadoop YARN > Issue Type: Sub-task > Reporter: Xuan Gong > Assignee: Xuan Gong > Attachments: YARN-7072-trunk.001.patch, YARN-7072.trunk.002.patch, > YARN-7072-trunk.003.patch, YARN-7072-trunk.004.patch, > YARN-7072-trunk.005.patch, YARN-7072-trunk.006.patch > > -- This message was sent by Atlassian JIRA (v6.4.14#64029) --------------------------------------------------------------------- To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org