[BUGFIX] argilla server: Prevent update dataset.updated_at when updating dataset.last_activity_at column

argilla-server/CHANGELOG.md

@@ -16,6 +16,12 @@ These are the section headers that we use:
 
 ## [Unreleased]()
 
+### Fixed
+
+- Fixed error updating dataset.updated_at when updating dataset.last_activity_at. ([#]())
+
+## [2.4.0](https://github.com/argilla-io/argilla/compare/v2.3.1...v2.4.0)
+
 ### Added
 
 - Added new `metadata` attribute for endpoints getting, creating and updating Datasets so now it is possible to store metadata associated to a dataset. ([#5586](https://github.com/argilla-io/argilla/pull/5586))

argilla-server/src/argilla_server/contexts/datasets.py

@@ -100,7 +100,12 @@
 
 async def _touch_dataset_last_activity_at(db: AsyncSession, dataset: Dataset) -> None:
     await db.execute(
-        sqlalchemy.update(Dataset).where(Dataset.id == dataset.id).values(last_activity_at=datetime.utcnow())
+        sqlalchemy.update(Dataset)
+        .where(Dataset.id == dataset.id)
+        .values(
+            last_activity_at=datetime.utcnow(),
+            updated_at=Dataset.__table__.c.updated_at,
+        )
     )
 
 

argilla-server/tests/unit/api/handlers/v1/test_responses.py

@@ -102,6 +102,7 @@ async def test_update_response(
             "updated_at": datetime.fromisoformat(resp_body["updated_at"]).isoformat(),
         }
 
+        await db.refresh(dataset)
         assert dataset.last_activity_at > dataset_previous_last_activity_at
         assert dataset.updated_at == dataset_previous_updated_at
 
@@ -421,6 +422,7 @@ async def test_delete_response(
         assert resp.status_code == 200
         assert (await db.execute(select(func.count(Response.id)))).scalar() == 0
 
+        await db.refresh(dataset)
         assert dataset.last_activity_at > dataset_previous_last_activity_at
         assert dataset.updated_at == dataset_previous_updated_at