Revert "AMBARI-11192 - The Default hdfs-site.xml Should Have Client Retry Logic Enabled For Rolling Upgrade (jonathanhurley)"

ambari-server/src/main/java/org/apache/ambari/server/checks/CheckDescription.java

@@ -31,8 +31,6 @@ public enum CheckDescription {
   CLIENT_RETRY(PrereqCheckType.SERVICE,
       "Client Retry Properties",
       new HashMap<String, String>() {{
-        put(ClientRetryPropertyCheck.HDFS_CLIENT_RETRY_MISSING_KEY,
-          "The hdfs-site.xml property dfs.client.retry.policy.enabled should be set to true.");
         put(ClientRetryPropertyCheck.HIVE_CLIENT_RETRY_MISSING_KEY,
           "The hive-site.xml property hive.metastore.failure.retries should be set to a positive value.");
         put(ClientRetryPropertyCheck.OOZIE_CLIENT_RETRY_MISSING_KEY,

ambari-server/src/main/java/org/apache/ambari/server/checks/ClientRetryPropertyCheck.java

@@ -39,7 +39,6 @@ import com.google.inject.Singleton;
 @UpgradeCheck(group = UpgradeCheckGroup.CLIENT_RETRY_PROPERTY)
 public class ClientRetryPropertyCheck extends AbstractCheckDescriptor {
 
-  static final String HDFS_CLIENT_RETRY_MISSING_KEY = "hdfs.client.retry.missing.key";
   static final String HIVE_CLIENT_RETRY_MISSING_KEY = "hive.client.retry.missing.key";
   static final String OOZIE_CLIENT_RETRY_MISSING_KEY = "oozie.client.retry.missing.key";
 
@@ -80,14 +79,9 @@ public class ClientRetryPropertyCheck extends AbstractCheckDescriptor {
 
     List<String> errorMessages = new ArrayList<String>();
 
+    // We excluded hdfs-site property dfs.client.retry.policy.enabled because default is false, and should remain
+    // that way due to *****.
     // check hdfs client property
-    if (services.containsKey("HDFS")) {
-      String hdfsClientRetry = getProperty(request, "hdfs-site", "dfs.client.retry.policy.enabled");
-      if (null == hdfsClientRetry || !Boolean.parseBoolean(hdfsClientRetry)) {
-        errorMessages.add(getFailReason(HDFS_CLIENT_RETRY_MISSING_KEY, prerequisiteCheck, request));
-        prerequisiteCheck.getFailedOn().add("HDFS");
-      }
-    }
 
     // check hive client properties
     if (services.containsKey("HIVE")) {

ambari-server/src/main/resources/stacks/HDP/2.2/services/HDFS/configuration/hdfs-site.xml

@@ -36,7 +36,7 @@
   
   <property>
     <name>dfs.client.retry.policy.enabled</name>
-    <value>true</value>
+    <value>false</value>
     <description>Enables HDFS client retry in the event of a NameNode failure.</description>
   </property>
 

ambari-server/src/test/java/org/apache/ambari/server/checks/ClientRetryPropertyCheckTest.java

@@ -113,18 +113,6 @@ public class ClientRetryPropertyCheckTest {
     m_check.perform(check, new PrereqCheckRequest("cluster"));
     Assert.assertEquals(PrereqCheckStatus.FAIL, check.getStatus());
 
-    // hdfs retry false
-    properties.put("dfs.client.retry.policy.enabled", "false");
-    check = new PrerequisiteCheck(null, null);
-    m_check.perform(check, new PrereqCheckRequest("cluster"));
-    Assert.assertEquals(PrereqCheckStatus.FAIL, check.getStatus());
-
-    // hdfs retry true
-    properties.put("dfs.client.retry.policy.enabled", "true");
-    check = new PrerequisiteCheck(null, null);
-    m_check.perform(check, new PrereqCheckRequest("cluster"));
-    Assert.assertEquals(PrereqCheckStatus.PASS, check.getStatus());
-
     // add hive
     services.put("HIVE", Mockito.mock(Service.class));