[ https://issues.apache.org/jira/browse/HADOOP-11293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14206236#comment-14206236 ]
Steve Loughran commented on HADOOP-11293: ----------------------------------------- ...and before anyone says its our own fault that SLIDER uses this in its production and test code, I've just checked and {{org.apache.hadoop.yarn.applications.distributedshell.ApplicationMaster}} uses it too. That is: the reference YARN AM makes use of the probe to build commands for different platforms. It is therefore effectively public, irrespective of what the @ scope tags say. > Factor OSType out from Shell > ---------------------------- > > Key: HADOOP-11293 > URL: https://issues.apache.org/jira/browse/HADOOP-11293 > Project: Hadoop Common > Issue Type: Improvement > Components: util > Reporter: Yongjun Zhang > Assignee: Yongjun Zhang > Attachments: HADOOP-11293.001.patch > > > Currently the code that detects the OS type is located in Shell.java. Code > that need to check OS type refers to Shell, even if no other stuff of Shell > is needed. > I am proposing to refactor OSType out to its own class, so to make the > OSType easier to access and the dependency cleaner. > -- This message was sent by Atlassian JIRA (v6.3.4#6332)