[ https://issues.apache.org/jira/browse/HADOOP-17111?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17150346#comment-17150346 ]
Ahmed Hussein commented on HADOOP-17111: ---------------------------------------- I will organize the imports in those source files and upload a new patch. I was reluctant to change the imports to minimize the diffs but I think you have a good point there in doing it in a clean way. Can you please review the patch 002 after the yetus report gets generated and commit it to trunk? Thanks [~aajisaka]! > Replace Guava Optional with Java8+ Optional > ------------------------------------------- > > Key: HADOOP-17111 > URL: https://issues.apache.org/jira/browse/HADOOP-17111 > Project: Hadoop Common > Issue Type: Sub-task > Reporter: Ahmed Hussein > Assignee: Ahmed Hussein > Priority: Major > Attachments: HADOOP-17111.001.patch > > > {code:java} > Targets > Occurrences of 'com.google.common.base.Optional' in project with mask > '*.java' > Found Occurrences (3 usages found) > org.apache.hadoop.yarn.server.nodemanager (2 usages found) > DefaultContainerExecutor.java (1 usage found) > 71 import com.google.common.base.Optional; > LinuxContainerExecutor.java (1 usage found) > 22 import com.google.common.base.Optional; > org.apache.hadoop.yarn.server.resourcemanager.recovery (1 usage found) > TestZKRMStateStorePerf.java (1 usage found) > 21 import com.google.common.base.Optional; > {code} -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org