better code

Author: hanfei1991

src/Parsers/FunctionSecretArgumentsFinder.h

@@ -623,11 +623,15 @@ class FunctionSecretArgumentsFinder
             /// PostgreSQL('host:port', 'database', 'user', 'password')
             findMySQLDatabaseSecretArguments();
         }
-        else if (engine_name == "S3" || engine_name == "DataLakeCatalog")
+        else if (engine_name == "S3")
         {
             /// S3('url', 'access_key_id', 'secret_access_key')
             findS3DatabaseSecretArguments();
         }
+        else if (engine_name == "DataLakeCatalog")
+        {
+            findDataLakeCatalogSecretArguments();
+        }
     }
 
     void findMySQLDatabaseSecretArguments()
@@ -658,6 +662,14 @@ class FunctionSecretArgumentsFinder
         }
     }
 
+    void findDataLakeCatalogSecretArguments()
+    {
+        /// datalake catalog should support different storage types,
+        /// we need a function to check if the url is S3 or Azure.
+        /// right now we assume it's a S3 url
+        findS3DatabaseSecretArguments();
+    }
+
     void findBackupNameSecretArguments()
     {
         const String & engine_name = function->name();