Doc for implementation details

getindata · Jun 8, 2023 · e321728 · e321728
1 parent 94647ca
commit e321728
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 1 deletion.
diff --git a/docs/source/06_mlflow.md b/docs/source/06_mlflow.md
@@ -4,6 +4,23 @@
 
 ![MLflow and Kedro-snowflake](../images/mlflow-support.png)
 
+## Implementation details
+Kedro-Snowflake <-> MLflow integration is based on the following concepts:
+ * [Snowflake external functions](https://docs.snowflake.com/en/sql-reference/external-functions-introduction) that
+are used for wrapping POST requests to the MLflow instance. In the minimal setup the following wrapping external functions for MLflow REST API calls must be created:
+    * [Create run](https://mlflow.org/docs/latest/rest-api.html#create-run)
+    * [Update run](https://mlflow.org/docs/latest/rest-api.html#update-run)
+    * [Log param](https://mlflow.org/docs/latest/rest-api.html#log-param)
+    * [Log metric](https://mlflow.org/docs/latest/rest-api.html#log-metric)  
+    * [Search experiment](https://mlflow.org/docs/latest/rest-api.html#search-experiments)
+ * [Snowflake externa function translators](https://docs.snowflake.com/en/sql-reference/external-functions-translators) for
+ changing the format of the data sent/received from the MLflow instance.
+ * [Snowflake API integration](https://docs.snowflake.com/en/sql-reference/sql/create-api-integration) for setting up
+ a communication channel from the Snowflake instance to the cloud HTTPS proxy/gateway service
+ where your MLflow instance is hosted (e.g. Amazon API Gateway, Google Cloud API Gateway or Azure API Management). 
+ * [Snowflake storage integration](https://docs.snowflake.com/en/sql-reference/sql/create-storage-integration) to enable
+ your Snowflake instance to upload artifacts (e.g. serialized models) to the cloud storage (Amazon S3, Azure Blob Storage, Google Cloud Storage) used by the
+ MLflow instance. 
 ## Configuration example
 
 ```yaml
@@ -15,7 +32,9 @@
     functions:
       experiment_get_by_name: demo.demo.mlflow_experiment_get_by_name
       run_create: demo.demo.mlflow_run_create
+      run_update: demo.demo.mlflow_run_update
       run_log_metric: demo.demo.mlflow_run_log_metric
       run_log_parameter: demo.demo.mlflow_run_log_parameter
-
 ```
+
+## Dedicated Kedro starter
diff --git a/kedro_snowflake/config.py b/kedro_snowflake/config.py
@@ -84,6 +84,7 @@ class SnowflakeRuntimeConfig(BaseModel):
 class MLflowFunctionsConfig(BaseModel):
     experiment_get_by_name: str = "mlflow_experiment_get_by_name"
     run_create: str = "mlflow_run_create"
+    run_update: str = "mlflow_run_update"
     run_log_metric: str = "mlflow_run_log_metric"
     run_log_parameter: str = "mlflow_run_log_parameter"
 
@@ -182,6 +183,7 @@ class KedroSnowflakeConfig(BaseModel):
     functions:
         experiment_get_by_name: mlflow_experiment_get_by_name
         run_create: mlflow_run_create
+        run_update: mlflow_run_update
         run_log_metric: mlflow_run_log_metric
         run_log_parameter: mlflow_run_log_parameter
 """.strip()

diff --git a/kedro_snowflake/starters/snowflights/{{ cookiecutter.repo_name }}/conf/base/snowflake.yml b/kedro_snowflake/starters/snowflights/{{ cookiecutter.repo_name }}/conf/base/snowflake.yml
@@ -73,5 +73,6 @@ snowflake:
     functions:
       experiment_get_by_name: mlflow_experiment_get_by_name
       run_create: mlflow_run_create
+      run_update: mlflow_run_update
       run_log_metric: mlflow_run_log_metric
       run_log_parameter: mlflow_run_log_parameter