Use underscores for semantic versions in table names (#1093)

* Use underscores for semantic versions in table names

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* fixup! [pre-commit.ci] auto fixes from pre-commit.com hooks

* fixup! [pre-commit.ci] auto fixes from pre-commit.com hooks

* fixup! [pre-commit.ci] auto fixes from pre-commit.com hooks

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Author: 0x2b3bfa0

src/datachain/data_storage/warehouse.py

@@ -258,7 +258,7 @@ def dataset_table_name(self, dataset_name: str, version: str) -> str:
         if Client.is_data_source_uri(dataset_name):
             # for datasets that are created for bucket listing we use different prefix
             prefix = self.DATASET_SOURCE_TABLE_PREFIX
-        return f"{prefix}{dataset_name}_{version}"
+        return f"{prefix}{dataset_name}_{version.replace('.', '_')}"
 
     def temp_table_name(self) -> str:
         return self.TMP_TABLE_NAME_PREFIX + _random_string(6)

tests/func/test_warehouse.py

@@ -1,6 +1,6 @@
 def test_dataset_stats_no_table(cloud_test_catalog, dogs_dataset):
     catalog = cloud_test_catalog.catalog
-    catalog.warehouse.drop_dataset_rows_table(dogs_dataset, 1)
-    num_objects, size = catalog.warehouse.dataset_stats(dogs_dataset, 1)
+    catalog.warehouse.drop_dataset_rows_table(dogs_dataset, version="1.0.0")
+    num_objects, size = catalog.warehouse.dataset_stats(dogs_dataset, version="1.0.0")
     assert num_objects is None
     assert size is None