add extra debug statements

Author: ciur

CHANGELOG.md

@@ -0,0 +1,5 @@
+# Changelog
+
+## 0.3.9 - 2024-04-21
+
+- Add more logging/debug statements

LICENSE

@@ -2,7 +2,7 @@
                            Version 2.0, January 2004
                         http://www.apache.org/licenses/
 
-   Copyright 2023 Eugen Ciur <[email protected]>
+   Copyright 2023-2024 Eugen Ciur <[email protected]>
 
    TERMS AND CONDITIONS FOR USE, REPRODUCTION, AND DISTRIBUTION
 

poetry.lock

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "salinic"
-version = "0.3.9"
+version = "0.3.10"
 description = "Search abstraction layer"
 authors = ["Eugen Ciur <[email protected]>"]
 readme = "README.md"

pyproject.toml

@@ -1,6 +1,7 @@
 import logging
 
 import requests
+from requests.exceptions import HTTPError
 
 from salinic.query import SearchQuery
 from salinic.url import URL
@@ -71,24 +72,52 @@ def remove(self, **kwargs):
         return response
 
     def update_schema(self, data):
-        return requests.post(
+        logger.debug(
+            f'POST {self.http_update_url} json={data}'
+        )
+        response = requests.post(
             self.http_schema_url,
             json=data
         )
+        response.raise_for_status()
 
     def field_exists(self, name: str) -> bool:
         # for normal fields
-        response = requests.get(self.http_field_url(name))
-
-        if response.status_code == 404:
+        url = self.http_field_url(name)
+        logger.debug(f'GET {url}')
+        response = requests.get(url)
+
+        try:
+            response.raise_for_status()
+        except HTTPError:
+            if response.status_code == 404:
+                # status 404 indicates that field does not exist, this
+                # is normal flow
+                logger.debug(f"Field {name} does not exist")
+                return False
+            # we got some other error messages (i.e. != 404)
+            # log it
+            logger.exception()
             return False
 
         return True
 
     def dynamicfield_exists(self, name: str) -> bool:
-        response = requests.get(self.http_dynamicfield_url(name))
-
-        if response.status_code == 404:
+        url = self.http_dynamicfield_url(name)
+        logger.debug(f'GET {url}')
+        response = requests.get(url)
+
+        try:
+            response.raise_for_status()
+        except HTTPError:
+            if response.status_code == 404:
+                # status 404 indicates that field does not exist, this
+                # is normal flow
+                logger.debug(f"Dynamic field {name} does not exist")
+                return False
+            # we got some other error messages (i.e. != 404)
+            # log it
+            logger.exception()
             return False
 
         return True

salinic/backends/solr/client.py

@@ -1,10 +1,13 @@
+import logging
 from typing import List, Tuple
 
 from salinic.field import Field, NumericField
 from salinic.utils import first
 
 from .types import CopyFieldDump, FieldDump, FieldType
 
+logger = logging.getLogger(__name__)
+
 
 class SchemaManager:
     def __init__(self, client, model):
@@ -15,7 +18,9 @@ def create(self):
         self.client.update_schema(self.create_dict_dump())
 
     def apply(self):
-        self.client.update_schema(self.apply_dict_dump())
+        dict_dump = self.apply_dict_dump()
+        logger.debug(f"Apply schema changes, dict_dump: {dict_dump}")
+        self.client.update_schema(dict_dump)
 
     def delete(self):
         self.client.update_schema(self.delete_dict_dump())