destroy: use dvc_object's remove() instead of shutil.rmtree

`dvc_objects.fs.utils.remove()` works for readonly files too.

src/datachain/cache.py

@@ -1,12 +1,12 @@
 import os
-import shutil
 from collections.abc import Iterator
 from contextlib import contextmanager
 from tempfile import mkdtemp
 from typing import TYPE_CHECKING, Optional
 
 from dvc_data.hashfile.db.local import LocalHashFileDB
 from dvc_objects.fs.local import LocalFileSystem
+from dvc_objects.fs.utils import remove
 from fsspec.callbacks import Callback, TqdmCallback
 
 from .progress import Tqdm
@@ -117,7 +117,7 @@ def clear(self):
 
     def destroy(self):
         # `clear` leaves the prefix directory structure intact.
-        shutil.rmtree(self.cache_dir)
+        remove(self.cache_dir)
 
     def get_total_size(self) -> int:
         total = 0

src/datachain/lib/file.py

@@ -279,10 +279,11 @@ async def _prefetch(self, catalog=None, download_cb=None) -> bool:
 
         client = catalog.get_client(self.source)
         if client.protocol == HfClient.protocol:
-            self._set_stream(catalog, self._caching_enabled, download_cb=download_cb)
             return False
 
+        download_cb = download_cb or self._download_cb
         await client._download(self, callback=download_cb)
+        # remove callback
         self._set_stream(catalog, caching_enabled=True)
         return True