Warn instead of error when col description provided for non-existent col

Author: treysp

sqlmesh/core/model/meta.py

@@ -249,7 +249,9 @@ def _column_descriptions_validator(
         if columns_to_types:
             for column_name in col_descriptions:
                 if column_name not in columns_to_types:
-                    raise ConfigError(
+                    from sqlmesh.core.console import get_console
+
+                    get_console().log_warning(
                         f"In model '{info.data['name']}', a description is provided for column '{column_name}' but it is not a column in the model."
                     )
 

tests/core/test_model.py

@@ -2907,7 +2907,8 @@ def a_model(context):
 
     assert py_model.columns_to_types.keys() == py_model.column_descriptions.keys()
 
-    # error: `columns` and `column_descriptions` column names are different cases, quoting preserves case
+    # warning: `columns` and `column_descriptions` column names are quoted and different cases,
+    #   so column not present in model
     @model(
         "col_descriptions_quoted",
         columns={'"col"': "int"},
@@ -2916,11 +2917,13 @@ def a_model(context):
     def b_model(context):
         pass
 
-    with pytest.raises(ConfigError, match="a description is provided for column 'COL'"):
+    with patch.object(get_console(), "log_warning") as mock_logger:
         py_model = model.get_registry()["col_descriptions_quoted"].model(
             module_path=Path("."),
             path=Path("."),
         )
+        assert mock_logger.call_count == 1
+        assert "a description is provided for column 'COL'" in mock_logger.call_args[0][0]
 
 
 def test_python_model_unsupported_kind() -> None: