address pr comments

Author: dwaijam

lang/cs/Org.Apache.REEF.IO.Tests/TestAzureBlockBlobFileSystemE2E.cs

@@ -36,7 +36,9 @@ namespace Org.Apache.REEF.IO.Tests
     /// </summary>
     public sealed class TestAzureBlockBlobFileSystemE2E : IDisposable
     {
-        private const string SkipMessage = "Fill in credentials before running test"; // Use null to run tests
+        // Uncomment SkipMessage = null to run tests
+        private const string SkipMessage = "Fill in credentials before running test";
+        // private const string SkipMessage = null;
         private const string HelloFile = "hello";
         private IFileSystem _fileSystem;
         private CloudBlobClient _client;

lang/cs/Org.Apache.REEF.IO/FileSystem/AzureBlob/AzureCloudBlobClient.cs

@@ -86,7 +86,6 @@ public BlobResultSegment ListBlobsSegmented(
                 continuationToken,
                 blobRequestOptions,
                 operationContext);
-            task.Wait();
             return task.Result;
         }
 

lang/cs/Org.Apache.REEF.IO/FileSystem/Hadoop/HadoopFileSystem.cs

@@ -17,7 +17,6 @@
 
 using System;
 using System.Collections.Generic;
-using System.Globalization;
 using System.IO;
 using System.Linq;
 using System.Text.RegularExpressions;
@@ -156,11 +155,7 @@ public bool IsDirectory(Uri uri)
         {
             // TODO[JIRA REEF - 2039]: HadoopFileSystem .IsDirectory() check needs to work on linux machines.
             var stdOut = _commandRunner.Run("dfs -test -d " + uri + "& call echo %^errorlevel%").StdOut;
-            if (stdOut.Any())
-            {
-                return "0".Equals(stdOut.First());
-            }
-            return false;
+            return stdOut.Any() && stdOut.First() == "0";
         }
 
         public void DeleteDirectory(Uri directoryUri)