[SPARK-19748][SQL]refresh function has a wrong order to do cache invalidate and regenerate the inmemory var for InMemoryFileIndex with FileStatusCache

sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InMemoryFileIndex.scala

@@ -66,8 +66,8 @@ class InMemoryFileIndex(
   }
 
   override def refresh(): Unit = {
-    refresh0()
     fileStatusCache.invalidateAll()
+    refresh0()
   }
 
   private def refresh0(): Unit = {

sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/FileIndexSuite.scala

@@ -178,6 +178,34 @@ class FileIndexSuite extends SharedSQLContext {
       assert(catalog2.allFiles().nonEmpty)
     }
   }
+
+  test("refresh for InMemoryFileIndex with FileStatusCache") {
+    withTempDir { dir =>
+      val fileStatusCache = FileStatusCache.getOrCreate(spark)
+      val dirPath = new Path(dir.getAbsolutePath)
+      val catalog = new InMemoryFileIndex(spark, Seq(dirPath), Map.empty,
+        None, fileStatusCache) {
+        def leafFilePaths: Seq[Path] = leafFiles.keys.toSeq
+        def leafDirPaths: Seq[Path] = leafDirToChildrenFiles.keys.toSeq
+      }
+
+      assert(catalog.leafDirPaths.isEmpty)
+      assert(catalog.leafFilePaths.isEmpty)
+
+      val file = new File(dir, "text.txt")
+      stringToFile(file, "text")
+
+      catalog.refresh()
+
+      assert(catalog.leafFilePaths.size == 1)
+      assert(catalog.leafFilePaths.head.toString.stripSuffix("/") ==
+        s"file:${file.getAbsolutePath.stripSuffix("/")}")
+
+      assert(catalog.leafDirPaths.size == 1)
+      assert(catalog.leafDirPaths.head.toString.stripSuffix("/") ==
+        s"file:${dir.getAbsolutePath.stripSuffix("/")}")
+    }
+  }
 }
 
 class FakeParentPathFileSystem extends RawLocalFileSystem {