fix(ingest): ensure that LookML files are always parsed in the same o…

…rder (#2966)
datahub-project · Jul 29, 2021 · acffd8f · acffd8f
1 parent e88ccd9
commit acffd8f
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/metadata-ingestion/src/datahub/ingestion/source/lookml.py b/metadata-ingestion/src/datahub/ingestion/source/lookml.py
@@ -139,8 +139,9 @@ def resolve_includes(
                 # Need to handle a relative path.
                 glob_expr = str(pathlib.Path(path).parent / inc)
             # "**" matches an arbitrary number of directories in LookML
-            outputs = glob.glob(glob_expr, recursive=True) + glob.glob(
-                f"{glob_expr}.lkml", recursive=True
+            outputs = sorted(
+                glob.glob(glob_expr, recursive=True)
+                + glob.glob(f"{glob_expr}.lkml", recursive=True)
             )
             if "*" not in inc and not outputs:
                 reporter.report_failure(path, f"cannot resolve include {inc}")